1231552 - backout Proxy: Firefox fallback to regular connection

Status: RESOLVED FIXED

(Core :: Networking, defect)

Description

[Daniel Stenberg [:bagder]]

Backout the changes for these bugs 

   Bug 1207798 - add pref to toggle proxy behavior when inaccessible. r=mcmanus
   Bug 1214200: avoid double NS_RELEASE(), r=mcmanus
   Bug 1121800 -  when all proxies fail try DIRECT, r=mcmanus

This will instead be handled as a slightly larger take in bug 1230803

Comment 1

[Daniel Stenberg [:bagder]]

Attachment: 0001-Bug-1231552-Back-out-bug-1121800-bug-1207798-and-bug.patch

Comment 2

[Daniel Stenberg [:bagder]]

Comment on attachment 8697127 [details] [diff] [review]
0001-Bug-1231552-Back-out-bug-1121800-bug-1207798-and-bug.patch

Approval Request Comment
[Feature/regressing bug #]: 1121800 and 1207798 
[User impact if declined]: changes proxy behavior in undesired ways, possibly leaking information 
[Describe test coverage new/current, TreeHerder]: lacking
[Risks and why]: the backout should be very small risk
[String/UUID change made/needed]:

Comment 3

[Carsten Book [:Tomcat]]

hi, this failed to apply:

applying 0001-Bug-1231552-Back-out-bug-1121800-bug-1207798-and-bug.patch
patching file modules/libpref/init/all.js
Hunk #1 FAILED at 1824
1 out of 1 hunks FAILED -- saving rejects to file modules/libpref/init/all.js.rej
patch failed, unable to continue (try -v)
patch failed, rejects left in working directory
errors during apply, please fix and qrefresh 0001-Bug-1231552-Back-out-bug-1121800-bug-1207798-and-bug.patch

Comment 4

[Daniel Stenberg [:bagder]]

Attachment: v2-0001-Bug-1231552-Back-out-bug-1121800-bug-1207798-and-.patch

Rebased, take 2.

The all.js file is a bit sensitive to modify as it is quite a popular thing... =)

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e8b02a6a50d2

Comment 6

[Daniel Stenberg [:bagder]]

Comment on attachment 8697265 [details] [diff] [review]
v2-0001-Bug-1231552-Back-out-bug-1121800-bug-1207798-and-.patch

Approval Request Comment
[Feature/regressing bug #]: Primarily the fix for bug 1121800
[User impact if declined]: We introduce a changed behavior that some people object to, due to the risk of leaking network information
[Describe test coverage new/current, TreeHerder]: lacking
[Risks and why]: just a reversion of three patches, low risk
[String/UUID change made/needed]:

Comment 7

[Guilherme Lima]

https://hg.mozilla.org/mozilla-central/rev/e8b02a6a50d2 - landed 15 hours ago.

Comment 8

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Sylvestre, we need to revert these patches in Aurora soon so we are safer by default and not bypassing proxy unless explicitly told/pref'd to do so (coming in a new fix). Could you please approve the Aurora uplift?

Comment 9

[Sylvestre Ledru [:Sylvestre]]

Comment on attachment 8697265 [details] [diff] [review]
v2-0001-Bug-1231552-Back-out-bug-1121800-bug-1207798-and-.patch

Sure, sorry, I was waiting for aurora update to be enabled!

Comment 10

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Tomcat, Wes: Please help land in m-a. Thanks!

Comment 11

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

These backouts landed in 45, which is what is now currently on aurora. Does this actually need to go to 44 (beta) instead?

Comment 12

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Let's ask Daniel. Is this a backout patch for 45 or 44? Please let us know.

Comment 13

[Daniel Stenberg [:bagder]]

When I requested this, aurora was still 44. It has now turned into beta.

The two commits that are still in beta that should get backed out to align it with this bug and the state in m-i:

https://hg.mozilla.org/releases/mozilla-beta/rev/13db8516d365
https://hg.mozilla.org/releases/mozilla-beta/rev/a62a3c113e40

The latter being a single-line fix of the first commit.

(bug 1207798 clearly never landed in 44)

Comment 14

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Comment on attachment 8697265 [details] [diff] [review]
v2-0001-Bug-1231552-Back-out-bug-1121800-bug-1207798-and-.patch

Approval Request Comment
See the earlier aurora approval request.

Comment 15

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Comment on attachment 8697265 [details] [diff] [review]
v2-0001-Bug-1231552-Back-out-bug-1121800-bug-1207798-and-.patch

We need to back this out from FF44, Beta+

Comment 16

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/releases/mozilla-beta/rev/9d310a41f8c2