Skip to content

Populate headers for HTTP OPTIONS requests #1062

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
gittiver merged 6 commits into from
Mar 23, 2026
Merged

Populate headers for HTTP OPTIONS requests #1062

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
2816221
Handle headers for HTTPMethod::Options
DavidPetkovsek Jul 16, 2025
a32a84f
Prevent memory leak from Options requests trying to upgrade into webs…
DavidPetkovsek Jul 16, 2025
1c37d57
Fix erroneous unit test
DavidPetkovsek Jul 27, 2025
6d83f6d
Fix erroneous unit test 2
DavidPetkovsek Jul 27, 2025
ee421d6
Exit early for requests with no route for optional too
DavidPetkovsek Jul 27, 2025
ff68f50
fix handle_header
DavidPetkovsek Mar 3, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 8 additions & 2 deletions include/crow/http_connection.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ namespace crow
{
routing_handle_result_ = handler_->handle_initial(req_, res);
// if no route is found for the request method, return the response without parsing or processing anything further.
if (!routing_handle_result_->rule_index && !routing_handle_result_->catch_all)
if (!routing_handle_result_->rule_index && !routing_handle_result_->catch_all && (req_.method != HTTPMethod::Options || routing_handle_result_->method == HTTPMethod::InternalMethodCount))
{
parser_.done();
need_to_call_after_handlers_ = true;
Expand All @@ -136,6 +136,12 @@ namespace crow
CROW_LOG_ERROR << ec << " buffer write error happened while handling sending continuation buffer header";
}
}
if (!routing_handle_result_->rule_index && !routing_handle_result_->catch_all && req_.method == HTTPMethod::Options)
{
parser_.done();
need_to_call_after_handlers_ = true;
complete_request();
}
}

void handle()
Expand All @@ -161,7 +167,7 @@ namespace crow
is_invalid_request = true;
res = response(400);
}
else if (req_.upgrade)
else if (req_.upgrade && req_.method != HTTPMethod::Options)
{
// h2 or h2c headers
if (req_.get_header_value("upgrade").find("h2")==0)
Expand Down
4 changes: 2 additions & 2 deletions tests/unittest.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1291,7 +1291,7 @@ TEST_CASE("middleware_cors")
CHECK(resp.find("Access-Control-Allow-Credentials: true") != std::string::npos);

resp = HttpClient::request(LOCALHOST_ADDRESS, port,
"OPTIONS /auth-origin / HTTP/1.1 \r\n\r\n");
"OPTIONS /auth-origin HTTP/1.1\r\n\r\n");
CHECK(resp.find("Access-Control-Allow-Origin: *") != std::string::npos);
CHECK(resp.find("Access-Control-Allow-Credentials: true") == std::string::npos);

Expand Down Expand Up @@ -2800,7 +2800,7 @@ TEST_CASE("option_header_passed_in_full")
};

std::string request =
"OPTIONS /echo HTTP/1.1\r\n";
"OPTIONS /echo HTTP/1.1\r\n\r\n";

auto res = make_request(request);
CHECK(res.find(ServerName) != std::string::npos);
Expand Down
Loading