79242 - IPv6: PAC: isInNet function needs to support new addressing

Status: NEW

(Core :: Networking: Proxy, defect, P5)

Description

[[e-mail]]

From bug 58030

------- Additional Comments From [e-mail] 2001-01-15 07:55 -------
+"function isInNet(ipaddr, pattern, maskstr) {\n"+
this _definitely_ doesn't look ipv6 friendly. :-(

Possible issues:

1- decision on whether we need a new function call or can make use of the
exsiting paratmeters)
2- support for the actual IPv6 addressing data types.

Comment 1

[benc]

qa to me. 

Comment 2

[neeti]

-->PAC bugs to Jussi

Comment 3

[Jussi-Pekka Mantere]

Serge, can you take a look at these PAC issues? Thanks - Jussi-Pekka

Comment 4

[Blake Ross]

Doesn't look like this is getting fixed before the freeze tonight.
Pushing out a milestone.  Please correct if I'm mistaken.

Comment 5

[serge (gone)]

this is an enhancement, removing TM.

Comment 6

[rubydoo123]

moving to future, enhancement requests will be reviewed at a later date

Comment 7

[bobbell]

I respectfully request that the status of this bug be reconsidered.  I believe
that it is a full-fledged bug, not a request for enhancement.

I am currently running Mozilla 1.1a on Tru64 UNIX V5.1B.  Mozilla is intended to
be my primary browser on my workstation.  However, this bug is preventing my
proxy configuration from working, forcing me to constantly diddle with the proxy
configuration myself or fall back to Netscape Navigator 4.7x.

It took me a long time to identify the issue, but eventually I was able to
determine that dnsResolve(hostname) was returning an IP address in the format
"::ffff:15.1.2.3".  This breaks isInNet, since convert_addr is splitting on ".",
and the first part thus becomes "::ffff:15", not a valid integer.  As a result,
isInNet() incorrectly returns false.

Note that this is not an issue on Netscape Navigator 4.7x.  Thus, this behavior
is a regression from previous behavior.  It is currently inhibiting my ability
to use Mozilla.

I would respectfully request that if the owner of this bug disagrees with my
assessment, that he say so, so that at least I know that my comments have been
addressed.  I will wait for the owners response, instead of changing the status
of the bug myself (which would probably be annoying for the owner).

Thank you.

Comment 8

[[e-mail]]

4xp.
putting into the triage queue w/ helpwanted.

Comment 9

[Doug Turner (:dougt)]

future

Comment 10

[Brant Gurganus]

[RFE] is deprecated in favor of severity: enhancement.  They have the same meaning.

Comment 11

[bobbell]

I'm sorry, but I'm still wondering why this is a RFE/enhancement bug.  I believe
it to be a bug in isInNet that is forcing me to revert to NN4 in some cases.

Comment 12

[Doug Turner (:dougt)]

okay.

Comment 13

[benc]

There is no explict support for IPv6 in the original specs. This is  "4xp", so
the fact that Comm somewhat does this correctly (and that this is to some degree
a product-level regresssion) is marked w/ that keyword.

Comment 14

[João Carlos Mendes Luís]

Bug 238341

Comment 15

[benc]

bobbell: 

I think Darin fixed the function so that it would stop breaking isInNet (it now
only returns IPv4 addresses).

Please let me know if this is fixed, if so, mark this RESOLVED/FIXED.

Comment 16

[bobbell]

I personally haven't seen this recently.  I think my configuration changed, but
I'd say to go ahead and mark this FIXED.  I can't do so because I'm not the
submitter.

Comment 17

[Eric Hopper]

This is still not really fixed.  isInNet should fully support IPv6 addresses or some new function should be added which does.  IMHO, the use of 'mask' is pretty IPv6 unfriendly.  Prefixes (i.e. addr/bits ala 192.168.43.0/24) is better suited for both IPv4 and IPv6.

Comment 18

[Christian Ertl]

As noted in Bug 242352, MS has already introduced additions for IPv6:

http://blogs.msdn.com/wndp/archive/2006/07/18/IPV6_WPAD_for_WinHttp_and_WinInet.aspx

Comment 19

[Firefox Bug Husbandry Bot]

Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258

Comment 20

[BMO Automation]

Moving bug to Core/Networking: Proxy.