Skip to content

Commit

Permalink
Merge branch 'master' of github.com:yhirose/cpp-httplib
Browse files Browse the repository at this point in the history
  • Loading branch information
yhirose committed Oct 18, 2024
2 parents 3701195 + d869054 commit fa90d06
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 2 deletions.
4 changes: 2 additions & 2 deletions httplib.h
Original file line number Diff line number Diff line change
Expand Up @@ -396,7 +396,7 @@ struct hash {
}
};

}; // namespace case_ignore
} // namespace case_ignore

// This is based on
// "http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2014/n4189".
Expand Down Expand Up @@ -4114,7 +4114,7 @@ inline bool parse_header(const char *beg, const char *end, T fn) {
p++;
}

if (p < end) {
if (p <= end) {
auto key_len = key_end - beg;
if (!key_len) { return false; }

Expand Down
9 changes: 9 additions & 0 deletions test/test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -4922,6 +4922,15 @@ TEST(ServerRequestParsingTest, InvalidFieldValueContains_CR_LF_NUL) {
EXPECT_EQ("HTTP/1.1 400 Bad Request", out.substr(0, 24));
}

TEST(ServerRequestParsingTest, EmptyFieldValue) {
std::string out;

test_raw_request("GET /header_field_value_check HTTP/1.1\r\n"
"Test: \r\n\r\n",
&out);
EXPECT_EQ("HTTP/1.1 200 OK", out.substr(0, 15));
}

TEST(ServerStopTest, StopServerWithChunkedTransmission) {
Server svr;

Expand Down

0 comments on commit fa90d06

Please sign in to comment.