Great, yes, this OPTIONS change is correct.

Background: OPTIONS didn't have a body in the past, but RFC 7231 mentions that 
it may have one for "future extensions". So the future is hereā€¦ 

btw., even Wikipedia interpreted it incorrectly as "No", I fixed it to 
"Optional":
https://en.wikipedia.org/w/index.php?title=Hypertext_Transfer_Protocol&stable=0#Summary_table

Reyk

> On 27.07.2016, at 12:46, Michael Lechtermann <bele...@bsdmail.de> wrote:
> 
> 
>> On 27Juli, 2016, at 12:26, Reyk Floeter <r...@openbsd.org> wrote:
>> 
>> better?
> 
> Yes, I could apply the patch now, however I had to make one additional 
> change. Now SVN appears to be working again.
> 
> # diff -u relay_http.c.orig relay_http.c
> --- relay_http.c.orig   Wed Jul 27 10:43:22 2016
> +++ relay_http.c        Wed Jul 27 10:43:54 2016
> @@ -376,12 +376,12 @@
>                case HTTP_METHOD_DELETE:
>                case HTTP_METHOD_GET:
>                case HTTP_METHOD_HEAD:
> -               case HTTP_METHOD_OPTIONS:
>                /* WebDAV methods */
>                case HTTP_METHOD_COPY:
>                case HTTP_METHOD_MOVE:
>                        cre->toread = 0;
>                        break;
> +               case HTTP_METHOD_OPTIONS:
>                case HTTP_METHOD_POST:
>                case HTTP_METHOD_PUT:
>                case HTTP_METHOD_RESPONSE:

Reply via email to