[PATCH] httprepo: long arguments support (issue2126)

[Laurens Holst]

Op 28-3-2011 19:38, Augie Fackler schreef:
> On Mon, Mar 28, 2011 at 12:13 PM, Laurens Holst<laurens.nospam at grauw.nl>  wrote:
>> Good point :).
>>
>> In that case I would suggest to just add custom headers instead of (abusing)
>> cookies for this...
>>
>> X-Mercurial-Changesets: 7248fd2710572839e3459d3d374eb9565edebbab,
>> b12142c8ba342fffac9280a528194fbec77c74fd,
>> 438fe4ee31b4531ee4a7f068d1affa402b1a3eb9
>>
>> Servers shouldn’t have a problem with that, and normal header length rules
>> should apply. You should also be able to add multiple headers of the same
>> name in case of comma separated values[1]. (To be a little briefer you could
>> also name the header X-Hg-Changeset.)
> +1, I've suggested this in the past and it sounds reasonable. We
> should also make sure that any response to a request that used headers
> sets appropriate cache-control headers to avoid potential GET caching
> issues.

Actually I think you should use the Vary header for that:

Vary: X-Hg-Changesets

Should do the trick.

~Laurens

-- 
~~ Ushiko-san! Kimi wa doushite, Ushiko-san nan da!! ~~
Laurens Holst, developer, Utrecht, the Netherlands
Website: www.grauw.nl. Backbase employee; www.backbase.com


-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 6034 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20110328/f35e9e63/attachment.bin>