[PATCH RFC] hgweb: add config option for maxhttpheadersize

Steven Brown stevengbrown at gmail.com
Wed May 4 10:45:49 CDT 2011


On 4 May 2011 04:47, Sune Foldager <cryo at cyanite.org> wrote:
> Does this look good to you? Setting it to 0, the new arguments-in-headers
> option can then be disabled, for debug or other purposes.
>
> -Sune

Yeah generally looks good. This is what I had in mind for a config option.

Comments:
- Why the "server" section, rather than "web" ?
- The tests are a little strange to read. I'd prefer see HGPORT1 with
error1.log, access1.log, and so on.
- Another test case would be a non-zero and non-default option, e.g.
10 characters.


More information about the Mercurial-devel mailing list