• 
      

    Wire up the target host for hosting service HTTPS requests.

    Review Request #15184 — Created July 21, 2026 and updated

    Information

    Review Board
    release-8.x

    Reviewers

    HostingServiceHTTPRequest.open() built a
    CertificateVerificationHTTPSHandler without telling it which server the
    request was for. The handler's hostname and port arguments 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_https failed before this
      change, asserting against a None context. It now passes.
    • Ran unit tests for reviewboard.certs and reviewboard.hostingsvcs.
    Summary ID
    Wire up the target host for hosting service HTTPS requests.
    `HostingServiceHTTPRequest.open()` built a `CertificateVerificationHTTPSHandler` without telling it which server the request was for. The handler's `hostname` and `port` arguments 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. Testing Done: - `BuildSSLContextFromSSLCertTests.test_open_with_https` failed before this change, asserting against a `None` context. It now passes. - Ran unit tests for `reviewboard.certs` and `reviewboard.hostingsvcs`.
    slxqyquloqsmtoqqtmwmnnsoqnpntntv
    Checks run (2 succeeded)
    flake8 passed.
    JSHint passed.