HTTPS proxying proxy auth (407) problems

RESOLVED FIXED in Firefox 32

Status

()

Core
Networking: HTTP
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: mcmanus, Assigned: mcmanus)

Tracking

33 Branch
mozilla33
x86_64
Linux
Points:
---

Firefox Tracking Flags

(firefox31 unaffected, firefox32 fixed, firefox33 fixed)

Details

(Whiteboard: [spdy][http2release])

Attachments

(4 attachments)

Comment hidden (empty)
(Assignee)

Comment 1

3 years ago
Created attachment 8453948 [details] [diff] [review]
part 1/3 dont doauthretry with closed persistent connection
Attachment #8453948 - Flags: review?(hurley)
(Assignee)

Comment 2

3 years ago
Created attachment 8453949 [details] [diff] [review]
part 2/3 407 proxy auth for http over https
Attachment #8453949 - Flags: review?(hurley)
(Assignee)

Comment 3

3 years ago
Created attachment 8453950 [details] [diff] [review]
part 3/3 407 proxy auth for https over https
Attachment #8453950 - Flags: review?(hurley)
(Assignee)

Updated

3 years ago
Whiteboard: [spdy][http2release]
(Assignee)

Comment 4

3 years ago
https://tbpl.mozilla.org/?tree=Try&rev=2fa3b4401cb3
Attachment #8453948 - Flags: review?(hurley) → review+
Attachment #8453949 - Flags: review?(hurley) → review+
Comment on attachment 8453950 [details] [diff] [review]
part 3/3 407 proxy auth for https over https

Review of attachment 8453950 [details] [diff] [review]:
-----------------------------------------------------------------

Side note: this looks like it will also fix an issue I noticed (but had not yet filed) while debugging bug 1014600 (namely that the squid error I get back when trying to connect to a host that doesn't answer never shows up). So, yay!
Attachment #8453950 - Flags: review?(hurley) → review+
(Assignee)

Updated

3 years ago
status-firefox31: --- → unaffected
status-firefox32: --- → affected
status-firefox33: --- → affected
(Assignee)

Comment 6

3 years ago
remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/01ccb9eab0f9
remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/697a57c606c4
remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/cb0ebf1d80ec
https://hg.mozilla.org/mozilla-central/rev/01ccb9eab0f9
https://hg.mozilla.org/mozilla-central/rev/697a57c606c4
https://hg.mozilla.org/mozilla-central/rev/cb0ebf1d80ec
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
(Assignee)

Comment 8

3 years ago
Created attachment 8456337 [details] [diff] [review]
gecko 32 version of patch

Approval Request Comment
[Feature/regressing bug #]:378637
[User impact if declined]: New feature does not do proxy auth correctly
[Describe test coverage new/current, TBPL]: on m-c, hand tested to same level as non https proxying
[Risks and why]: risks are constrained to new feature which is on aurora - feature has broken corner case without patch
[String/UUID change made/needed]: none
Attachment #8456337 - Flags: review+
Attachment #8456337 - Flags: approval-mozilla-aurora?
(Assignee)

Updated

3 years ago
status-firefox33: affected → fixed
Although this only landed on m-c yesterday, I'm not sure that giving this another day is going to buy us anything in terms of additional testing. I don't see any tests in the patch. Are there existing automated tests for this functionality?
Flags: needinfo?(mcmanus)
(Assignee)

Comment 10

3 years ago
unfortunately we don't have any automated proxy tests in our CI at all - and this feature is an extension of that capability, so it had to be tested by hand right now. Building something is on my todo list, but its considerably bigger than this fix, or even the feature it fixes.
Flags: needinfo?(mcmanus)
Comment on attachment 8456337 [details] [diff] [review]
gecko 32 version of patch

Aurora+

Patrick - Understandable about automating tests for this feature. Can you please run your manual sanity checks on Aurora once this lands?
Attachment #8456337 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
(Assignee)

Comment 12

3 years ago
remote:   https://hg.mozilla.org/releases/mozilla-aurora/rev/abc10706ecbb
(Assignee)

Updated

3 years ago
status-firefox32: affected → fixed
(Assignee)

Comment 13

3 years ago
(In reply to Lawrence Mandel [:lmandel] from comment #11)
> . Can you
> please run your manual sanity checks on Aurora once this lands?

yes - I will. I've also already manually checked the patch on a local build of the aurora tree.
(Assignee)

Comment 14

3 years ago
(In reply to Lawrence Mandel [:lmandel] from comment #11)
 Can you
> please run your manual sanity checks on Aurora once this lands?

done
You need to log in before you can comment on or make changes to this bug.