Re: [xml] nanohttp: No redirects for http://www.domain.com



On Fri, Oct 02, 2009 at 04:49:10PM +0200, Ralf Junker wrote:
At 15:42 02.10.2009, Daniel Veillard wrote:

  Yes I tend to agree, this is an oversight.
And I think this affect only the path part, for other parts like
fragment, query or username, there is an extra symbol (# ? or @)
which would get lost if we didn't convert to "" .

I have sketched up a few basic tests and the other pieces seem to work 
fine, i.e. they are indeed NULL and not an empty string.

But for all the case for path I think this need fixing, could you try 
the enclosed patch which I think covers all cases ?

The patch works fine for the issue reported: xmlParseURIRaw() returns  
expected values and nanohttp also redirects as usual.

  Okidoc, commited, thanks for double-checking !

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel veillard com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]