[PATCH] httprepo: long arguments support (issue2126)

Dirkjan Ochtman dirkjan at ochtman.nl
Sun Mar 20 10:52:11 CDT 2011


On Sun, Mar 20, 2011 at 16:27, Matt Mackall <mpm at selenic.com> wrote:
> One of the reasons we were focusing on stashing stuff in headers is that
> some folks might be relying on non-push requests being GETs in their web
> server access rules. I'm not sure if this is a real problem, though.
> Obviously, a POST approach is a lot cleaner.

Meh, I don't like it. I guess our protocol is already more RPC-like
than REST-like, but I would rather strive to keep idempotent requests
as GET and use POST only for the unbundle command.

I'm also confused on what this is supposed to fix. The message
mentions issue2126, but messages there seem to declare that we only
have an actual problem (left) on unbundle, not on any of the GET
commands. I guess the issue might be that we're trying to reduce the
number of round trips needed because of an excessive list of heads?

Cheers,

Dirkjan


More information about the Mercurial-devel mailing list