Skip Menu |
 

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

Report information
The Basics
Id: 21411
Status: resolved
Priority: 0/
Queue: HTTP-Server-Simple

People
Owner: Nobody in particular
Requestors: junior [...] sixapart.com
Cc:
AdminCc:

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



Subject: header regex invalid
Date: Sun, 10 Sep 2006 00:24:02 -0700
To: bug-HTTP-Server-Simple [...] rt.cpan.org
From: Mark Smith <junior [...] sixapart.com>
Download (untitled) / with headers
text/plain 458b
Version 0.20 of HTTP::Server::Simple in sub parse_headers only allows a very restricted set of headers. This is not enough to fully express valid HTTP headers. Notably, a period is allowable in HTTP headers, but gets ignored by the module as invalid. (And as an aside, the right side is allowed to be blank, which is also excluded here.) It may be worth expanding this function to allow the full range of HTTP headers. -- Mark Smith junior@sixapart.com
Download (untitled) / with headers
text/plain 679b
On Sun Sep 10 03:24:23 2006, junior@sixapart.com wrote: Show quoted text
> Version 0.20 of HTTP::Server::Simple in sub parse_headers only allows
a Show quoted text
> very restricted set of headers. > > This is not enough to fully express valid HTTP headers. Notably, a
period Show quoted text
> is allowable in HTTP headers, but gets ignored by the module as
invalid. Show quoted text
> (And as an aside, the right side is allowed to be blank, which is also > excluded here.) > > It may be worth expanding this function to allow the full range of
HTTP Show quoted text
> headers. > > > -- > Mark Smith > junior@sixapart.com
Hi Mark, As of svn r12129, the regex is now: /^([^()<>\@,;:\\"\/\[\]?={} \t]+):\s*(.*)/i That should meet RFC 2616. Shawn


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.