Skip Menu |
 

This queue is for tickets about the HTTP-Message CPAN distribution.

Report information
The Basics
Id: 59570
Status: resolved
Priority: 0/
Queue: HTTP-Message

People
Owner: Nobody in particular
Requestors: GUIDO [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: (no value)
Fixed in: (no value)



Subject: HTTP::Request->parse() handles leading double slashes in path incorrectly
Download (untitled) / with headers
text/plain 1.2k
use strict; use URI; use HTTP::Request; my $request = HTTP::Request->parse('GET //foo/bar/baz?uvw#xyz HTTP/1.1'); warn $request->uri->authority; my $uri = URI->new('//foo/bar/baz?uvw#xyz'); warn $uri->authority; __END__ Output is "foo" both times. With libwww-perl 5.827 and URI 1.38. The problem is caused by URI being too relaxed about the format of a URI. But I guess that cannot be fixed because it would break a lot of existing code. I think HTTP::Request->parse must be changed. The Request-URI of the Request-Line in HTTP/1.1 is either a *, an absolute URI, an absolute path, or an authority. Unfortunately my version of URI is too smart for being fixed here: my $uri = URI->new; $uri->path('//foo/bar/baz'); warn $uri->authority; This spits out a friendly warning that a leading double slash is confusing, and it then happily forgets that this is just the path, not an authority. In fact, a leading double slash in a path is not really confusing but it is perfecly legal and all browsers send such requests, and they conform to HTTP here. I have no idea how this could be fixed in a clean way without giving up the blessed-scalar-approach for URI. But it basically renders the HTTP::Request->parse() method useless. Cheers, Guido


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.