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)



MIME-Version: 1.0
X-Spam-Status: No, hits=-2.6 required=8.0 tests=BAYES_00
Content-Disposition: inline
Received-SPF: pass (x1.develooper.com: local policy)
content-type: text/plain; charset="utf-8"
Received: from la.mx.develooper.com (ss1.fabel.dk [63.251.223.179]) by diesel.bestpractical.com (Postfix) with SMTP id 0B9304D80CB for <bug-HTTP-Server-Simple [...] rt.cpan.org>; Sun, 10 Sep 2006 03:24:17 -0400 (EDT)
Received: (qmail 3031 invoked by alias); 10 Sep 2006 07:24:16 -0000
Received: from Unknown (HELO mail.xb95.com) (207.7.148.202) by la.mx.develooper.com (qpsmtpd/0.28) with SMTP; Sun, 10 Sep 2006 00:24:08 -0700
Received: (qmail 25821 invoked by uid 1000); 10 Sep 2006 07:24:02 -0000
Delivered-To: cpan-bug+http-server-simple [...] diesel.bestpractical.com
Subject: header regex invalid
User-Agent: Mutt/1.5.11+cvs20060403
Return-Path: <mark [...] xb95.com>
X-Original-To: bug-HTTP-Server-Simple [...] rt.cpan.org
X-Spam-Check-BY: la.mx.develooper.com
Date: Sun, 10 Sep 2006 00:24:02 -0700
Message-Id: <20060910072402.GA7510 [...] plogs.net>
To: bug-HTTP-Server-Simple [...] rt.cpan.org
From: Mark Smith <junior [...] sixapart.com>
X-RT-Original-Encoding: us-ascii
Content-Length: 458
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
MIME-Version: 1.0
In-Reply-To: <20060910072402.GA7510 [...] plogs.net>
X-Mailer: MIME-tools 5.418 (Entity 5.418)
Content-Disposition: inline
Charset: utf8
References: <20060910072402.GA7510 [...] plogs.net>
Message-Id: <rt-3.6.HEAD-25343-1210148452-1706.21411-0-0 [...] rt.cpan.org>
Content-Type: text/plain
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Original-Encoding: utf-8
Content-Length: 679
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.