From 0a063062438e283c3c717de13060d5cb128d4be2 Mon Sep 17 00:00:00 2001
From: Yurun <admin@yurunsoft.com>
Date: Wed, 14 Jul 2021 15:50:16 +0800
Subject: [PATCH 1/2] Fix the problem of parsing http protocol when the request
 header lacks spaces

---
 src/plugins/http/inc/httpParser.h |  1 +
 src/plugins/http/src/httpParser.c | 15 ++++++++++++++-
 2 files changed, 15 insertions(+), 1 deletion(-)

diff --git a/src/plugins/http/inc/httpParser.h b/src/plugins/http/inc/httpParser.h
index c7cf7a9608..cabd1693a0 100644
--- a/src/plugins/http/inc/httpParser.h
+++ b/src/plugins/http/inc/httpParser.h
@@ -100,6 +100,7 @@ typedef enum HTTP_PARSER_STATE {
   HTTP_PARSER_CHUNK,
   HTTP_PARSER_END,
   HTTP_PARSER_ERROR,
+  HTTP_PARSER_OPTIONAL_SP,
 } HTTP_PARSER_STATE;
 
 typedef enum HTTP_AUTH_TYPE {
diff --git a/src/plugins/http/src/httpParser.c b/src/plugins/http/src/httpParser.c
index a68a9bd7ed..ba88a2b9cd 100644
--- a/src/plugins/http/src/httpParser.c
+++ b/src/plugins/http/src/httpParser.c
@@ -744,6 +744,15 @@ static int32_t httpParserOnSp(HttpParser *parser, HTTP_PARSER_STATE state, const
   return ok;
 }
 
+static int32_t httpParserOnOptionalSp(HttpParser *parser, HTTP_PARSER_STATE state, const char c, int32_t *again) {
+  int32_t ok = 0;
+  if (c != ' ') {
+    *again = 1;
+    httpPopStack(parser);
+  }
+  return ok;
+}
+
 static int32_t httpParserOnStatusCode(HttpParser *parser, HTTP_PARSER_STATE state, const char c, int32_t *again) {
   HttpContext *pContext = parser->pContext;
   int32_t      ok = 0;
@@ -867,7 +876,7 @@ static int32_t httpParserOnHeader(HttpParser *parser, HTTP_PARSER_STATE state, c
       }
       httpPushStack(parser, HTTP_PARSER_CRLF);
       httpPushStack(parser, HTTP_PARSER_HEADER_VAL);
-      httpPushStack(parser, HTTP_PARSER_SP);
+      httpPushStack(parser, HTTP_PARSER_OPTIONAL_SP);
       httpPushStack(parser, HTTP_PARSER_HEADER_KEY);
       break;
     }
@@ -1061,6 +1070,10 @@ static int32_t httpParseChar(HttpParser *parser, const char c, int32_t *again) {
       ok = httpParserOnSp(parser, state, c, again);
       break;
     }
+    if (state == HTTP_PARSER_OPTIONAL_SP) {
+      ok = httpParserOnOptionalSp(parser, state, c, again);
+      break;
+    }
     if (state == HTTP_PARSER_STATUS_CODE) {
       ok = httpParserOnStatusCode(parser, state, c, again);
       break;

From fb92d7b988678c977deb03b3d8bb77f4addc9b49 Mon Sep 17 00:00:00 2001
From: Yurun <admin@yurunsoft.com>
Date: Fri, 16 Jul 2021 16:31:45 +0800
Subject: [PATCH 2/2] remove the last comma

---
 src/plugins/http/inc/httpParser.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/plugins/http/inc/httpParser.h b/src/plugins/http/inc/httpParser.h
index cabd1693a0..865d275874 100644
--- a/src/plugins/http/inc/httpParser.h
+++ b/src/plugins/http/inc/httpParser.h
@@ -100,7 +100,7 @@ typedef enum HTTP_PARSER_STATE {
   HTTP_PARSER_CHUNK,
   HTTP_PARSER_END,
   HTTP_PARSER_ERROR,
-  HTTP_PARSER_OPTIONAL_SP,
+  HTTP_PARSER_OPTIONAL_SP
 } HTTP_PARSER_STATE;
 
 typedef enum HTTP_AUTH_TYPE {