[Varnish] #1801: Fail parsing IPv6 constants

Varnish varnish-bugs at varnish-cache.org
Tue Oct 20 16:09:19 CEST 2015


#1801: Fail parsing IPv6 constants
----------------------+-----------------------
 Reporter:  fgsch     |       Owner:
     Type:  defect    |      Status:  reopened
 Priority:  normal    |   Milestone:
Component:  varnishd  |     Version:  4.1.0
 Severity:  normal    |  Resolution:
 Keywords:            |
----------------------+-----------------------
Changes (by fgsch):

 * status:  closed => reopened
 * resolution:  wontfix =>


Comment:

 I don't agree with the resolution of this.

 To begin with, the rfc1884 (2001:db8::1) format used to work in previous
 versions.
 The new required format works for IP literals but doesn't work as input
 for std.ip().

 Obviously when ports are needed using []s is the only way, for anything
 else this is forced and arbitrary, specially considering how short and
 trivial the patch to support the other notation is (attached).

 We don't force the IPv4 literal to be IPv4:PORT so I don't see why we need
 to force IPv6 to be that way. This is not consistent either.

 We ought to reconsider it.

-- 
Ticket URL: <https://www.varnish-cache.org/trac/ticket/1801#comment:4>
Varnish <https://varnish-cache.org/>
The Varnish HTTP Accelerator



More information about the varnish-bugs mailing list