Skip to content

Commit 6d8cf82

Browse files
author
marius
committed
modified FSM so that the whitespace between header-key and value is optional
1 parent 48be5a5 commit 6d8cf82

File tree

2 files changed

+2
-5
lines changed

2 files changed

+2
-5
lines changed

include/server/request_parser.hpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,6 @@ class RequestParser
6161
header_line_start,
6262
header_lws,
6363
header_name,
64-
space_before_header_value,
6564
header_value,
6665
expecting_newline_2,
6766
expecting_newline_3

src/server/request_parser.cpp

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -217,7 +217,7 @@ RequestParser::RequestStatus RequestParser::consume(http::request &current_reque
217217
case internal_state::header_name:
218218
if (input == ':')
219219
{
220-
state = internal_state::space_before_header_value;
220+
state = internal_state::header_value;
221221
return RequestStatus::indeterminate;
222222
}
223223
if (!is_char(input) || is_CTL(input) || is_special(input))
@@ -226,14 +226,12 @@ RequestParser::RequestStatus RequestParser::consume(http::request &current_reque
226226
}
227227
current_header.name.push_back(input);
228228
return RequestStatus::indeterminate;
229-
case internal_state::space_before_header_value:
229+
case internal_state::header_value:
230230
if (input == ' ')
231231
{
232232
state = internal_state::header_value;
233233
return RequestStatus::indeterminate;
234234
}
235-
return RequestStatus::invalid;
236-
case internal_state::header_value:
237235
if (input == '\r')
238236
{
239237
state = internal_state::expecting_newline_2;

0 commit comments

Comments
 (0)