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)

History Show all quoted text
  Sun Sep 10 03:24:23 2006 junior [...] sixapart.com - Ticket created
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
  Wed May 07 04:09:50 2008 rt.cpan [...] sartak.org - Broken in 0.33 added
  Wed May 07 04:20:52 2008 rt.cpan [...] sartak.org - Correspondence added
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
  Wed May 07 04:20:53 2008 The RT System itself - Status changed from 'new' to 'open'
  Wed May 07 04:20:58 2008 rt.cpan [...] sartak.org - Status changed from 'open' to 'resolved'