Closed Bug 1383748 Opened 5 years ago Closed 5 years ago

https proxy does not work properly


(Core :: Networking, defect)

55 Branch
Not set



Tracking Status
firefox-esr52 --- unaffected
firefox54 --- unaffected
firefox55 - fixed
firefox56 --- fixed


(Reporter: Off.Just.Off, Assigned: wcpan)



(Keywords: regression, Whiteboard: [necko-active])


(1 file)

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:54.0) Gecko/20100101 Firefox/54.0
Build ID: 20170710123658

Steps to reproduce:

Setup https proxy using nsIProtocolProxyService and try to use it.

Here is the test add-on:

Proxy used in this example is restricted by domain list, so we should go to test url, that is known as allowed to access through this proxy:

Actual results:

Firefox 55+ can't establish connection through the proxy.

Expected results:

Browser should receive the answer from the test server through the proxy.
Component: Untriaged → Networking
Product: Firefox → Core
[Tracking Requested - why for this release]: https proxy does not work properly

It seems to trigger this problem by your patch.
Could you look this?
Blocks: 1310127
Has Regression Range: --- → yes
Flags: needinfo?(wpan)
Keywords: regression
Assignee: nobody → wpan
Flags: needinfo?(wpan)
Thank you for digging into it, Wei-Cheng Pan!

I hope there is still a chance the fix will be included into upcoming 55 release.
Whiteboard: [necko-active]
Ever confirmed: true
Comment on attachment 8890816 [details]
Bug 1383748 - Fix SPDY while using proxy.

as its a regression, please fill out the uplift request
Attachment #8890816 - Flags: review?(mcmanus) → review+
Pushed by
Fix SPDY while using proxy. r=mcmanus
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
Comment on attachment 8890816 [details]
Bug 1383748 - Fix SPDY while using proxy.

Approval Request Comment
[Feature/Bug causing the regression]: bug 1310127
[User impact if declined]: HTTP Proxies might not work properly.
[Is this code covered by automated tests?]: No.
[Has the fix been verified in Nightly?]: Yes.
[Needs manual test from QE? If yes, steps to reproduce]: No.
[List of other uplifts needed for the feature/fix]: None.
[Is the change risky?]: No.
[Why is the change risky/not risky?]: This is a simple fix, does not affect other functions.
[String changes made/needed]: None.
Attachment #8890816 - Flags: approval-mozilla-beta?
Comment on attachment 8890816 [details]
Bug 1383748 - Fix SPDY while using proxy.

proxy/spdy fix for 55 rc1
Attachment #8890816 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Many thanks to everyone involved in fixing this issue and especially for the well-timed uplifting!
You need to log in before you can comment on or make changes to this bug.