Hi,

On Mon, 2008-10-20 at 09:12 +0200, Hannes Magnusson wrote:
> > I suggest extending the behaviour of header() so that when the first
> > argument does not contain a colon (and does not begin with "HTTP/"), e.g.
> > header('Vary'), it unsets the header with the specified name. I decided to
> > extend the existing header() function rather than adding a new function,
> > because the existing function already supports multiple actions (add, append
> > and replace).
> 
> 
> That isn't very intuitive. I would think it was a typo when reading
> such code and fix the header line...
> I'd suggest explicit header_remove("Vary");

Agreed, a specific function with a clear name sounds better.

Christian, can you update the patch? - thanks!

johannes



-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to