Wire up the target host for hosting service HTTPS requests.
Review Request #15184 — Created July 21, 2026 and discarded — Latest diff uploaded
HostingServiceHTTPRequest.open()built a
CertificateVerificationHTTPSHandlerwithout telling it which server the
request was for. The handler'shostnameandportarguments therefore had
no callers, and the handler was left without an SSL context until
https_open()built one from the request URL.These are now passed in from the request's URL, so the context is available
as soon as the handler is constructed.
BuildSSLContextFromSSLCertTests.test_open_with_httpsfailed before this
change, asserting against aNonecontext. It now passes.- Ran unit tests for
reviewboard.certsandreviewboard.hostingsvcs.