Re: [PATCH] http-backend: provide Allow header for 405

2013-09-10 Thread brian m. carlson
On Sun, Sep 08, 2013 at 09:18:12PM -0700, Jonathan Nieder wrote: > > --- a/http-backend.c > > +++ b/http-backend.c > > @@ -594,8 +594,11 @@ int main(int argc, char **argv) > > > > if (strcmp(method, c->method)) { > > const char *proto = getenv("SERV

Re: [PATCH] http-backend: provide Allow header for 405

2013-09-09 Thread Jeff King
On Sun, Sep 08, 2013 at 06:15:06PM +, brian m. carlson wrote: > The HTTP 1.1 standard requires an Allow header for 405 Method Not Allowed: > > The response MUST include an Allow header containing a list of valid methods > for the requested resource. > > So provide such a header when we r

Re: [PATCH] http-backend: provide Allow header for 405

2013-09-08 Thread Jonathan Nieder
Hi, brian m. carlson wrote: > --- a/http-backend.c > +++ b/http-backend.c > @@ -594,8 +594,11 @@ int main(int argc, char **argv) > > if (strcmp(method, c->method)) { > const char *proto = getenv("SERVER_PROTOCOL"); > -

[PATCH] http-backend: provide Allow header for 405

2013-09-08 Thread brian m. carlson
The HTTP 1.1 standard requires an Allow header for 405 Method Not Allowed: The response MUST include an Allow header containing a list of valid methods for the requested resource. So provide such a header when we return a 405 to the user agent. Signed-off-by: brian m. carlson --- http-back