• An HTTP request with responses: 200, 404.
• An HTTP request with a redirect
• 2 HTTP requests to the same origin over an HTTP/2 proxy.
  Check that the proxy connection is reused.
• 2 HTTP requests to the same origin over an HTTP/2 proxy. One of the requests uses the anonymous flag.
  Check that the proxy connection is not reused.
• 2 HTTP requests to the same origin over an HTTP/2 proxy with a different triggering principal.
  Check that the proxy connection is not reused.
• A connection to the proxy fails with a non-authentication failure. (test at least 2 HTTP requests).
• Test a long post and check progress notifications as well.
• Test a post with a slower writer.
• Test a long response.
• Test a slove response reader.
• Test canceling an HTTP request:
  • as soon as the nsHttpChannel is open and a connection to the proxy does not exist already,
  • as soon as the nsHttpChannel is open and a connection to the proxy exists (only HTTP/2 proxy),
  • after the request is sent but before the response is received
  • while receiving a response.
    For the above tests check that the HTTP/2 connection to the proxy is not closed.
• Network connection breaks with different errors in different stages of a request and proxy setup.

Also AltSvc is disabled for HTTP proxies - addapt the test.

The following tests should be covered with currently submitted patches or already existing tests:

• [x] An HTTP request with responses: 200, 404.
• [ ] An HTTP request with a redirect
• [x] 2 HTTP requests to the same origin over an HTTP/2 proxy.
  Check that the proxy connection is reused.
• [x] 2 HTTP requests to the same origin over an HTTP/2 proxy. One of the requests uses the anonymous flag.
  Check that the proxy connection is not reused.
• [x] 2 HTTP requests to the same origin over an HTTP/2 proxy with a different triggering principal.
  Check that the proxy connection is not reused.
• [x] A connection to the proxy fails with a non-authentication failure. (test at least 2 HTTP requests).
• [partially, only for HTTP2 server] Test a long post and check progress notifications as well.
• [ ] Test a post with a slower writer.
• [x] Test a long response.
• [x] Test a slove response reader.
• [ ] Test canceling an HTTP request:
  • as soon as the nsHttpChannel is open and a connection to the proxy does not exist already,
  • as soon as the nsHttpChannel is open and a connection to the proxy exists (only HTTP/2 proxy),
  • after the request is sent but before the response is received
  • while receiving a response.
    For the above tests check that the HTTP/2 connection to the proxy is not closed.
• [ ] Network connection breaks with different errors in different stages of a request and proxy setup.

The patch also changes test_server.js to await server shutdown when necessary.

We move from http.get to http.request to be able to pass the method to
the proxied request.
We alsoread the data from the stream and pipe it to the new request.

Depends on D155334

https://hg.mozilla.org/mozilla-central/rev/cd1684b91384
https://hg.mozilla.org/mozilla-central/rev/8a9725c09d40
https://hg.mozilla.org/mozilla-central/rev/06b90196355e

Backed out changeset 60fc8d3f8152 (Bug 1755660) for causing failures in test_proxy_cancel.js CLOSED TREE

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&group_state=expanded&collapsedPushes=842382&selectedTaskRun=FhLAKqCHSR2aGFLx-apk-w.0&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=60fc8d3f815251c9f8186e8c565b0665b84f898e

Log: https://treeherder.mozilla.org/logviewer?job_id=390546772&repo=autoland&lineNumber=4245

Backout: https://hg.mozilla.org/integration/autoland/rev/5910daede0a2de60e2c851b73122934e1e13fc16

https://hg.mozilla.org/mozilla-central/rev/ac2c4ef5bde1

Hi Kershaw, my appolgies but may I ask you to land 2 patches from me? I will not have time to fix them. Thank you

https://hg.mozilla.org/mozilla-central/rev/c1a7df2f329c

https://hg.mozilla.org/mozilla-central/rev/f36319780923

I assume we can close this now?

Yes, all tests are landed.