nsIPluginManager::FindProxyForURL isn't aware of "bypass proxies for local addresses"

VERIFIED FIXED in M18

Status

()

Core
Plug-ins
P3
normal
VERIFIED FIXED
19 years ago
18 years ago

People

(Reporter: edburns, Assigned: edburns)

Tracking

Trunk
All
Other
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [nsbeta2-])

Attachments

(1 attachment)

(Assignee)

Description

19 years ago
This bug is an upshoot from bug
http://bugzilla.mozilla.org/show_bug.cgi?id=15607

The problem is that FindProxyForURL isn't aware of the "bypass proxies for
local hosts" feature.

However, I don't know how this relates to "auto-configuration".  Robert Yang,
could you please elaborate?

Thanks,

ed
(Assignee)

Updated

19 years ago
Blocks: 15607

Comment 1

19 years ago
Under NS 4.7 proxy preferences one can observe the following :

1. Direct connection to internet.
2. Manual proxy connection.
3. Automatic proxy configuration URL.

Under item 2 (manual proxy configuration) there is a textbox for "Do not use
proxy servers for domains beginning with :"  This is the proxy exclusion list.
Addresses which begin with these names are not to be proxied.

Under item 3 (Automatic proxy configuration) there is a textbox for the URL of
a .pac file which contains javascript code as per the following :

http://home.netscape.com/eng/mozilla/2.0/relnotes/demo/proxy-live.html

This code is then used to determine what the proxy is.  Please let me know if
more info is needed.
(Assignee)

Updated

19 years ago
Depends on: 17158
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 2

19 years ago
Created a new bug for this, and made 16677 depend on it.  17158

Comment 3

19 years ago
Shrirang is now QA owner for Plug-ins; QA assigning all of my Plug-ins bugs over
to him.

Comment 4

18 years ago
This is fairly important.  If this bug is not fixed, a corporate user can run
Java applets from outside their corporate network, but can't run any applets
originating internally!!
(Assignee)

Comment 5

18 years ago
Nominating for nsbeta2+
Keywords: nsbeta2
Target Milestone: --- → M18

Comment 6

18 years ago
Putting on [nsbeta2-] radar. Not critical to beta2.  Adding "nsbeta3" keyword 
for consideration of a fix for that milestone. 
Keywords: nsbeta3
Whiteboard: [nsbeta2-]
(Assignee)

Comment 7

18 years ago
ETA 22 July 2000 
(Assignee)

Comment 8

18 years ago
Use user_pref("network.proxy.no_proxies_on", "arago.eng.sun.com, 
javaweb.eng.sun.com");
(Assignee)

Comment 9

18 years ago
Test code:

  // debug: edburns: test: start
  char *proxy;
  rv = this->FindProxyForURL("http://javaweb.eng.sun.com/", &proxy);
  printf("+++++debug: edburns: javaweb.eng.sun.com: FindProxyForURL: %s\n",
         proxy);
  nsCRT::free(proxy);
  rv = this->FindProxyForURL("http://javaweb/", &proxy);
  printf("+++++debug: edburns: javaweb: FindProxyForURL: %s\n",
         proxy);
  nsCRT::free(proxy);
  rv = this->FindProxyForURL("http://javaweb.eng/", &proxy);
  printf("+++++debug: edburns: javaweb.eng: FindProxyForURL: %s\n",
         proxy);
  nsCRT::free(proxy);
  rv = this->FindProxyForURL("http://www.suck.com/", &proxy);
  printf("+++++debug: edburns: www.suck.com: FindProxyForURL: %s\n",
         proxy);
  nsCRT::free(proxy);
  // debug: edburns: test: end
(Assignee)

Comment 10

18 years ago
Created attachment 11588 [details] [diff] [review]
Fix for this bug, first iteration.
(Assignee)

Comment 11

18 years ago
av, I'm looking for a= and r= on this.  Thanks.
(Assignee)

Comment 12

18 years ago
Checked in fix, marking fixed.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Comment 13

18 years ago
marking verified (rubberstamping)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.