GNU bug report logs - #10109
[PATCH] (web http): list-style headers do not validate

Previous Next

Full log

Message #16 received at 10109 <at> debbugs.gnu.org (full text, mbox):

From: Andy Wingo <wingo <at> pobox.com>
To: Daniel Hartwig <mandyke <at> gmail.com>
Cc: 10109 <at> debbugs.gnu.org
Subject: Re: bug#10109: [PATCH] (web http): list-style headers do not validate
Date: Thu, 22 Dec 2011 08:21:06 -0500

On Sun 27 Nov 2011 10:11, Daniel Hartwig <mandyke <at> gmail.com> writes:

> My apologies for not noticing earlier, but I have spotted a couple
> minor issues with both the previous patch and original code that are
> corrected by the attached. All relate to the "Cache-Control" header:
>
> - `max-stale' has optional value (previous code requires it)
> - some directives do not have values (`no-store', etc.)
> - there are `cache-extension' directives that may or may not have a value
>
> Attached patch tidies this up

Thanks, it was a great patch.  Applied and pushed.

> with explicit validation of all defined
> directives, though it leaves open one issue with the cache-extension
> directives:

I fixed this one, I think.

Happy hacking,

Andy
-- 
http://wingolog.org/

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.