jdrosa Reporter
	Description • 11 years ago

User agent: Mozilla/5.0 (Windows NT 6.1; rv:18.0) Gecko/20100101 Firefox/18.0

Build ID: 20130104151925

After automatically being upgraded to Firefox 18 from Firefox 17, all connectivity to internet sites, while being connected to an intranet, was lost. My browsers (I use Firefox 99% of the time, but am forced to use IE on occasion) are all configured to use a Proxy Auto Config (PAC) script to determine how to access internal vs. external websites.

Within Firefox, I had the PAC configured in the "Automatic proxy configuration URL" field. I then changed my Firefox configuration to the "Use system proxy settings," with no change in behavior. I could access intranet sites, but received the "Server not found" response whenever attempting to connect to external sites.

I then manually configured the proxy settings to force the use of the proxy server and I regained access to external sites, but then lost connectivity to internal sites.

I then placed a copy of the PAC on my local hard drive and reconfigured Firefox  to use the PAC on my local hard drive. Once again, no external connectivity.

All the while I was able to use IE and Chrome (after downloading it, I hate IE) to access internal and external sites using the non-local PAC URL.

Attached is a copy of the PAC.

	Patrick McManus [:mcmanus] Assignee
	Comment 1 • 11 years ago

The problem is 
var index = myIpAddress().lastIndexOf(".") + 1;
var myiparray = myIpAddress().split(".");

when myIPAddress() is executed outside of the context of calling FindProxyForURL() (in this case during pac file load time) it fails.

that's my bug. sorry.

your workaround is just to move those global lines into FindProxyForURL() and life should be good.

	Patrick McManus [:mcmanus] Assignee
	Comment 2 • 11 years ago

one other note - myIPAddress() is actually somewhat dependent on the host being looked up in FindProxyForURL().. in the case you have more than one IP address it will return the one that would be used to directly connect to that host... that's where the conflict came in.

I'll make it deal with the discrepancy, but you'll get a more accurate answer if your pac code scopes myIPAddress inside FindProxyForURL()

	Patrick McManus [:mcmanus] Assignee
	Comment 4 • 11 years ago

remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/c5bb3265e36e

	jdrosa Reporter
	Comment 5 • 11 years ago

I don't own the PAC script, but I will pass that recommendation on to the people that do.

Thanks for looking into this. I really hate using IE and Chrome just irritates me, so not having FF to use just ruins my day.

(In reply to Patrick McManus [:mcmanus] from comment #2)
> one other note - myIPAddress() is actually somewhat dependent on the host
> being looked up in FindProxyForURL().. in the case you have more than one IP
> address it will return the one that would be used to directly connect to
> that host... that's where the conflict came in.
> 
> I'll make it deal with the discrepancy, but you'll get a more accurate
> answer if your pac code scopes myIPAddress inside FindProxyForURL()

	Ryan VanderMeulen [:RyanVM]
	Comment 6 • 11 years ago

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

	Alex Keybl [:akeybl]
	Comment 7 • 11 years ago

Patrick - please help us understand the likelihood that this is a commonly occurring bug for proxy users on FF18, and the risk of regression if uplifted.

Please also prepare uplifts to FF19/20 asap.

	Patrick McManus [:mcmanus] Assignee
	Comment 8 • 11 years ago

(In reply to Alex Keybl [:akeybl] from comment #7)
> Patrick - please help us understand the likelihood that this is a commonly
> occurring bug for proxy users on FF18, and the risk of regression if
> uplifted.
> 

A PAC file is an arbitrary piece of javascript and this bug covers one such file.. obviously there an infinite number of possible pac configurations - its really hard to know how common this would be other than to say
 * only 1 person has reported the problem
 * the issue is with the global context and anecdotally most people just define 1 function (the required FindProxyForURL()) in their pac. But because its an arbitrary script its really hard to say

> Please also prepare uplifts to FF19/20 asap.

its the first thing I'll do after dropping my kid at school this morning. The fix is very eay to reason about in terms of safety.

	Patrick McManus [:mcmanus] Assignee
	Comment 9 • 11 years ago

Comment on attachment 701212 [details] [diff] [review]
patch 0

[Approval Request Comment]
Regression caused by (bug #): 769764
User impact if declined: myIpAddress() PAC helper function fails when used in top lovel scope. User can work around it if and only if they control the contents of the PAC file they are using.
Testing completed (on m-c, etc.): on m-c
Risk to taking this patch (and alternatives if risky): low - small fix.

	Patrick McManus [:mcmanus] Assignee
	Comment 10 • 11 years ago

remote:   https://hg.mozilla.org/releases/mozilla-aurora/rev/c238a35cfba8
remote:   https://hg.mozilla.org/releases/mozilla-beta/rev/442ebc2c2285

	Patrick McManus [:mcmanus] Assignee
	Comment 11 • 11 years ago

  https://hg.mozilla.org/releases/mozilla-release/rev/8c903e7225ee

	raymond [:retornam] (inactive)
	Comment 12 • 11 years ago

Checked in 18.0 on Mac OS X using the pac file here, editing the proxy addresses with a different proxy. It didn't work in 18.0. It works with 18.0.1.

	Ada [:adalucinet]
	Comment 14 • 11 years ago

There is an automated test in the patch that verifies this isssue, test_protocolproxyservice.js, and it passes on all platforms on beta and release channels.
Beta
https://tbpl.mozilla.org/php/getParsedLog.php?id=18881178&full=1&branch=mozilla-beta
https://tbpl.mozilla.org/php/getParsedLog.php?id=18877136&full=1&branch=mozilla-beta
https://tbpl.mozilla.org/php/getParsedLog.php?id=18878399&full=1&branch=mozilla-beta
Release
https://tbpl.mozilla.org/php/getParsedLog.php?id=18845657&full=1&branch=mozilla-release
https://tbpl.mozilla.org/php/getParsedLog.php?id=18843941&full=1&branch=mozilla-release
https://tbpl.mozilla.org/php/getParsedLog.php?id=18848657&full=1&branch=mozilla-release

	Ada [:adalucinet]
	Comment 15 • 11 years ago

This issue is verified on Firefox 19 beta 6 (Build ID: 20130212082553), on the next machines:

1) Ubuntu 12.04 64-bit
2) Windows 7 64-bit
3) Windows 8 32-bit
4) Mac OSX 10.6.8

I've verified it by changing all the proxies from the end of the attached pac file with  81.196.129.33:8080 and it works for me.