walkero
/
nghttp2
1
0
Fork 0
Code Issues Pull Requests Projects Releases 1 Wiki Activity

nghttpx: Use 502 as server error code

This commit is contained in:
Tatsuhiro Tsujikawa 2017-04-01 14:04:55 +09:00
parent b12c2a13c0
commit e17a6b29b6
4 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/shrpx_http2_upstream.cc
View File

@ -449,7 +449,7 @@ void Http2Upstream::initiate_downstream(Downstream *downstream) {
if (rv == SHRPX_ERR_TLS_REQUIRED) {
rv = redirect_to_https(downstream);
} else {
rv = error_reply(downstream, 503);
rv = error_reply(downstream, 502);
}
if (rv != 0) {
rst_stream(downstream, NGHTTP2_INTERNAL_ERROR);
@ -464,7 +464,7 @@ void Http2Upstream::initiate_downstream(Downstream *downstream) {
rv = downstream->attach_downstream_connection(std::move(dconn));
if (rv != 0) {
// downstream connection fails, send error page
if (error_reply(downstream, 503) != 0) {
if (error_reply(downstream, 502) != 0) {
rst_stream(downstream, NGHTTP2_INTERNAL_ERROR);
}
@ -477,7 +477,7 @@ void Http2Upstream::initiate_downstream(Downstream *downstream) {
rv = downstream->push_request_headers();
if (rv != 0) {
if (error_reply(downstream, 503) != 0) {
if (error_reply(downstream, 502) != 0) {
rst_stream(downstream, NGHTTP2_INTERNAL_ERROR);
}
@ -2005,7 +2005,7 @@ fail:
if (rv == SHRPX_ERR_TLS_REQUIRED) {
rv = on_downstream_abort_request_with_https_redirect(downstream);
} else {
rv = on_downstream_abort_request(downstream, 503);
rv = on_downstream_abort_request(downstream, 502);
}
if (rv != 0) {
rst_stream(downstream, NGHTTP2_INTERNAL_ERROR);

2
src/shrpx_http_downstream_connection.cc
View File

@ -152,7 +152,7 @@ void backend_retry(Downstream *downstream) {
if (rv == SHRPX_ERR_TLS_REQUIRED) {
rv = upstream->on_downstream_abort_request_with_https_redirect(downstream);
} else {
rv = upstream->on_downstream_abort_request(downstream, 503);
rv = upstream->on_downstream_abort_request(downstream, 502);
}
if (rv != 0) {

6
src/shrpx_https_upstream.cc
View File

@ -631,7 +631,7 @@ int HttpsUpstream::on_read() {
status_code = downstream->response().http_status;
if (status_code == 0) {
if (downstream->get_request_state() == Downstream::CONNECT_FAIL) {
status_code = 503;
status_code = 502;
} else if (downstream->get_request_state() ==
Downstream::HTTP1_REQUEST_HEADER_TOO_LARGE) {
status_code = 431;
@ -1310,7 +1310,7 @@ int HttpsUpstream::on_downstream_reset(Downstream *downstream, bool no_retry) {
// We have got all response body already. Send it off.
return 0;
case Downstream::INITIAL:
if (on_downstream_abort_request(downstream_.get(), 503) != 0) {
if (on_downstream_abort_request(downstream_.get(), 502) != 0) {
return -1;
}
return 0;
@ -1348,7 +1348,7 @@ fail:
if (rv == SHRPX_ERR_TLS_REQUIRED) {
rv = on_downstream_abort_request_with_https_redirect(downstream);
} else {
rv = on_downstream_abort_request(downstream_.get(), 503);
rv = on_downstream_abort_request(downstream_.get(), 502);
}
if (rv != 0) {
return -1;

2
src/shrpx_spdy_upstream.cc
View File

@ -1382,7 +1382,7 @@ int SpdyUpstream::on_downstream_reset(Downstream *downstream, bool no_retry) {
return 0;
fail:
if (on_downstream_abort_request(downstream, 503) != 0) {
if (on_downstream_abort_request(downstream, 502) != 0) {
rst_stream(downstream, SPDYLAY_INTERNAL_ERROR);
}
downstream->pop_downstream_connection();