1045640 - Disable HTTP Proxy over HTTPS on Gecko 32

Status: RESOLVED FIXED

(Core :: Networking: HTTP, defect)

Description

[Patrick McManus [:mcmanus]]

Attachment: BO

from bug 378637 comments 54 and 55


Approval Request Comment
[Feature/regressing bug #]: this one
[User impact if declined]: new feature that has reliability bugs related only to use of the feature (i.e. it is not generally destabilizing)
[Describe test coverage new/current, TBPL]: backout
[Risks and why]: minimal
[String/UUID change made/needed]: none

I'm going to ask that we disable this on FF32 via flipping an existing pref and instead target FF33. There is no UI - this is just new syntax in the PAC file format that won't be supported yet.

FF32 and FF33 are missing the related fixes for 1040930, 1040323, and 1043402. The first is on FF34 (and will be uplifted to 33) the latter two haven't landed at all yet. They're all expected to be small and suitable for FF33 but FF32 is too aggressive at this stage.

If it goes worse than planned we have plenty of time to disable on FF33, but this feature is eagerly awaited to give parity with Google's Compressing Data Proxy and it is receiving positive feedback.

-- comment 55 lmandel ---

OK. We'll back this out as per your suggestion. Can you please file a separate bug for the backout as that is much easier to track.

Comment 1

[Lawrence Mandel [:lmandel] (use needinfo)]

Comment on attachment 8464020 [details] [diff] [review]
BO

beta+

Comment 2

[Patrick McManus [:mcmanus]]

remote:   https://hg.mozilla.org/releases/mozilla-beta/rev/97a654ff8682

Comment 3

[Ed Morley [:emorley]]

This caused test failures across many suites:
https://tbpl.mozilla.org/?tree=Mozilla-Beta&rev=97a654ff8682

Would you mind backing it out? (My beta clone is broken)

Comment 4

[Patrick McManus [:mcmanus]]

(In reply to Ed Morley [:edmorley] from comment #3)
> This caused test failures across many suites:
> https://tbpl.mozilla.org/?tree=Mozilla-Beta&rev=97a654ff8682
> 
> Would you mind backing it out? (My beta clone is broken)

of course. Then I need to figure out why my smoke test was fine.

Comment 5

[Ryan VanderMeulen [:RyanVM]]

Allow me. Backed out for mass bustage.
https://hg.mozilla.org/releases/mozilla-beta/rev/433a7a6a2c25

https://tbpl.mozilla.org/php/getParsedLog.php?id=44865109&tree=Mozilla-Beta

Comment 6

[Patrick McManus [:mcmanus]]

Attachment: disable tls proxying bug 378637 on gecko 32

Comment 7

[Patrick McManus [:mcmanus]]

Comment on attachment 8469717 [details] [diff] [review]
disable tls proxying bug 378637 on gecko 32


this patch is a one line change to the previous pref backout - the pref switch was broken which is why this failed last time. The patch is r+'d in 1050598. This is very very safe.

I also wrote a test for it - unfortunately it exposed 2 other latent bugs in reconfiguration of PAC that are totally unrelated to the feature being disabled here. So I am omitting the test here, but will be commiting it to m-c along with the fixes to make it work. That's also bug 1050598

Comment 8

[Lawrence Mandel [:lmandel] (use needinfo)]

Comment on attachment 8469717 [details] [diff] [review]
disable tls proxying bug 378637 on gecko 32

OK. Let's give this another go. beta+

Comment 9

[Patrick McManus [:mcmanus]]

remote:   https://hg.mozilla.org/releases/mozilla-beta/rev/c9dec07b4148

Comment 10

[xunxun]

You should also change the 32 beta note: 
https://www.mozilla.org/en-US/firefox/32.0beta/releasenotes/

Comment 11

[Sylvestre Ledru [:Sylvestre]]

xunxun, done in the initial bug. I left it for 32 beta because it was enabled for some part of the beta cycle.