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

Report information
The Basics
Id:
21411
Status:
resolved
Priority:
Low/Low

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

BugTracker
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>
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
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 runs on Request Tracker, is sponsored by The Perl Foundation, and maintained by Best Practical Solutions.

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