Re: [PATCH] MINOR: http: Add OPTIONS in supported http methods (found by, find_http_meth)

[Willy Tarreau]

On Thu, Oct 08, 2015 at 02:03:19PM +0200, Christopher Faulet wrote:
> Hi,
> 
> The 'OPTIONS' method was not in the list of supported HTTP methods and 
> find_http_meth return HTTP_METH_OTHER instead of HTTP_METH_OPTIONS.

Wow good catch, I feel ashamed or having missed it. It was already
bogus in 1.3! I've merged it and tagged it as BUG/MINOR.

Thanks!
Willy

[PATCH] MINOR: http: Add OPTIONS in supported http methods (found by, find_http_meth)

[Christopher Faulet]

Hi,

The 'OPTIONS' method was not in the list of supported HTTP methods and 
find_http_meth return HTTP_METH_OTHER instead of HTTP_METH_OPTIONS.


Regards
--
Christopher Faulet
>From ec4669273b06bee074389baa2d00ef0202dbea1c Mon Sep 17 00:00:00 2001
From: Christopher Faulet <cfau...@qualys.com>
Date: Fri, 31 Jul 2015 14:26:57 +0200
Subject: [PATCH] MINOR: http: Add OPTIONS in supported http methods (found by
 find_http_meth)

---
 src/proto_http.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/proto_http.c b/src/proto_http.c
index 6d4a6b3..32b9063 100644
--- a/src/proto_http.c
+++ b/src/proto_http.c
@@ -458,6 +458,9 @@ const struct http_method_desc http_methods[26][3] = {
 	['H' - 'A'] = {
 		[0] = {	.meth = HTTP_METH_HEAD, .len=4, .text="HEAD"},
 	},
+	['O' - 'A'] = {
+		[0] = {	.meth = HTTP_METH_OPTIONS , .len=7, .text="OPTIONS" },
+	},
 	['P' - 'A'] = {
 		[0] = {	.meth = HTTP_METH_POST, .len=4, .text="POST"},
 		[1] = {	.meth = HTTP_METH_PUT , .len=3, .text="PUT" },
-- 
2.4.3