Closed
Bug 91121
Opened 23 years ago
Closed 9 years ago
PAC: "PrefConfig is not defined"
Categories
(Core :: Networking, defect)
Core
Networking
Tracking
()
RESOLVED
WONTFIX
Future
People
(Reporter: tingley, Unassigned)
References
()
Details
(Keywords: helpwanted, relnote)
Attachments
(2 files)
21.04 KB,
text/plain
|
Details | |
699 bytes,
patch
|
Details | Diff | Splinter Review |
Split off from bug 79893: ------- Additional Comments From Steve Chapel 2001-07-17 09:06 ------- When I try to use my company's PAC script, I get this error in the JavaScript Console: Error: PrefConfig is not defined Source File: http://config.mcgraw-hill.com.:11111/CONFIG/autoadmin.cgi Line: 211 The first three lines of the PAC script are: * Generated by Netscape Configuration Editor*/ with (PrefConfig) {
Reporter | ||
Comment 1•23 years ago
|
||
steven_chapel@mcgraw-hill.com, can you attach the PAC file for reference? I can't load it from the url you provided. Hmm, apparently this is some NN arcana: http://help.netscape.com/kb/consumer/19971009-14.html My understanding is that we're only planning to support a subset of the PAC stuff found in NN4 -- namely the stuff in http://home.netscape.com/eng/mozilla/2.0/relnotes/demo/proxy-live.html benc (or anyone else), can you provide some historical perspective?
Blocks: 79893
QA Contact: benc → pacqa
Comment 2•23 years ago
|
||
This file went into another feature that supported several newtork-based configurations, including PAC. This file is not valid in PAC, becasue PAC is a file that delivers ONLY the FindProxyforURL() function.
Comment 4•23 years ago
|
||
Hmmm... What should I do with that configuration file? In IE 6, I have that set as my "automatic configuration script" (my instructions from McGraw-Hill call it the "automatic proxy configuration" and "automatic proxy settings") and it works fine. In Mozilla, only the "manual proxy settings" works, but I lose some important functionality when I use the manual rather than the automatic settings. Should I open an evangelism bug on the script?
Reporter | ||
Comment 5•23 years ago
|
||
In the meantime, you can make a local file on your hard drive that contains just the FindProxyforURL() function the autoadmin script defines, and then set the autoconfig url to that file. ... Obviously, this isn't much help if the autoconfig function changes frequently.
It's another old feature of Communicator, where you could have lots of stuff globbed into a .jsc file. I don't know what the future of this feature is, so yeah, lets send it to evangelism or browser-general and see where it goes.
Component: Networking → Browser-General
a hallway conversation w/ an oldtimer produced an intersting tidbit: it is possible that the PAC loader in Communicator was modified so it did accept these files, by silently ignoring non-pac releated data. reporter: Does this file work if you configure the PAC setting in Communicator 4?
Comment 8•23 years ago
|
||
Whew! That's a blast from the past! I've never used Netscape 4 at work, so I just downloaded it and tried it. When Netscape 4 starts up, it tries to open the JavaScript with a helper app. So, no it doesn't work. When I set the PAC to the file that I created that has just the JS function the PAC does have, it does work. But that's downloaded from a file: URL, not an http: URL. When I try to load the entire PAC file from a file: URL, Netscape gives an error about the wrong MIME type. So it looks like it's the MIME type that's preventing it from working. If someone wants to talk me through step-by-step on how to set the PAC and run more tests, go ahead! I'm afraid I haven't memorized the W3C Specs. All I know is that when I follow the instructions I've been given, it works in IE6, but not in Netscape or Mozilla. Pretty ironic, since the comment shows that a Netscape program generated the PAC file in the first place! :-)
Reporter | ||
Comment 10•23 years ago
|
||
It does seem possible to throw enough stubs into the pacUtils to make this work -- I got a simplified version of the mcgraw-hill jsc file to act like a PAC (mozilla successfully loaded and used its FindProxyForURL implementation) by hacking around a bit. I can post a patch later on as an example. Whether we actually want to support this is another matter.
Comment 11•23 years ago
|
||
-> networking. I think we should consider isolating just the FindProxyforURL function, and ignoring the rest. The PAC specs say that the response should contain just the function, but that didn't help here :)
Component: Browser-General → Networking
Comment 12•23 years ago
|
||
As I understand the attached "PAC script" is file generated from netscape.cfg by netscape configuration editor, the old CCK tool. So it is not pure pac file, it is config file for communicator4.x which can handle js extensions like with (PrefConfig) {} or with (ProxyConfig) {}
Reporter | ||
Comment 13•23 years ago
|
||
Reporter | ||
Comment 14•23 years ago
|
||
The PAC sandbox can be trivially extended with enough stubs that the jsc file doesn't keel over when it loads. The PAC code can then extract the FindProxyForURL function normally. The attached patch was enough to get a trimmed down version of the mcgraw-hill file to load and execute. There may be other stubs needed to "support" all the things that could show up in a jsc file. This is pretty geezly, and I won't be surprised if people don't pursue it. It muddies the sandbox namespace and clutters things up in general, plus there may be some case I don't know about where this doesn't work.
Comment 16•23 years ago
|
||
RENOTE TEXT: "PrefConfig is not supported in this release. PAC files are a component of this, but the PAC setting will not accept PrefConfig URLs."
Comment 17•23 years ago
|
||
+nsenterprise - we need to find out how they are going to handle compatability of these features.
Keywords: nsenterprise
Comment 18•23 years ago
|
||
marking nsenterprise-; will be reevaluated for nsenterprise in future release.
Keywords: nsenterprise-
Comment 20•22 years ago
|
||
Temporarily "futuring" all PAC&SOCKS bugs to clear new-networking queue. I will review later. (I promise) If you object, and can make a case for a mozilla 1.0 fix, please reset milestone to "--" or email me.
Target Milestone: --- → Future
Updated•22 years ago
|
Keywords: helpwanted
Updated•15 years ago
|
Assignee: general → nobody
QA Contact: pacqa → networking
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•