GNU bug report logs -
#10410
guile: uri module confused by domain names starting with numbers, ipv6 addresses
Previous Next
Reported by: Daniel Hartwig <mandyke <at> gmail.com>
Date: Fri, 30 Dec 2011 10:18:02 UTC
Severity: normal
Tags: patch
Found in version 2.0.3
Done: Andy Wingo <wingo <at> pobox.com>
Bug is archived. No further changes may be made.
Full log
Message #22 received at 10410 <at> debbugs.gnu.org (full text, mbox):
On 6 July 2012 19:15, Andy Wingo <wingo <at> pobox.com> wrote:
> On Wed 22 Feb 2012 07:15, Daniel Hartwig <mandyke <at> gmail.com> writes:
>
>> Bumping this bug report as it is relevant to commit 1868309 which
>> recently addressed the issue with domain names starting with numbers.
>
> Sorry for the long delay.
Please, no :-) Your meticulous approach to maintenance is
commendable. All are confident of the eventuality of the next “serial
processing” session.
> I have applied your patches on my hasty
> commits. There were a couple bugs that this brought out; for example on
> my system, inet-pton does not expect the address to be surrounded by
> [].
Nice spot. Evidently when I originally looked in to this I did not
get much past string->uri and assumed the job done (*what* happened to
the actual task I was trying to complete then?!)
> I fixed this to have the uri-host / build-uri #:host to never have
> brackets.
Works well. One small nitpick, depending on what uri-host is defined
to be returning this may or may not differ from spec.:
host = IP-literal / IPv4address / reg-name
IP-literal = "[" ( IPv6address / IPvFuture ) "]"
In my intuition, a call to build-uri is naturally of the form:
(build-uri #:scheme 'http
#:host "[2001:db8::1]")
Though, for convenience, this begs for a predicate or two for
ip-literals and conversion from ip-literal->ipv6-address.
Anyway, working is better than not working, thanks :-)
This bug report was last modified 12 years and 318 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.