commit ef5418c52f115a85492dbc6b93e743669c56de1a
author Piotr Sikora <piotrsikora@google.com> Mon Oct 22 21:26:18 2018 -0700
committer Piotr Sikora <piotrsikora@google.com> Thu Oct 25 18:24:21 2018 +0000
tree d09d3e0ace80bf14f1d11be9d42e9be3e6e7acec
parent a2aa503f7a9b428576bb82e194bc96c3c88563e6

HTTP: change default response code when rate-limiting requests.

Revert default response code back to "503 Service Unavailable", because
the local change to "429 Too Many Requests" was never accepted upstream,
so it's an unnecessary incompatibility with the vanilla NGINX.

The old behavior can be restored using "limit_req_status" directive.

This reverts commit d4e033db526b0b5b0c75fd7d96e77a21f51667ad.

Change-Id: Ie0a39d63e3f354540fe71683c8373e2b4f002854
Signed-off-by: Piotr Sikora <piotrsikora@google.com>
Reviewed-on: https://nginx-review.googlesource.com/c/3520

src/http/modules/ngx_http_limit_req_module.c

diff --git a/src/http/modules/ngx_http_limit_req_module.c b/src/http/modules/ngx_http_limit_req_module.c
index 6648c4d..63ec2de 100644
--- a/src/http/modules/ngx_http_limit_req_module.c
+++ b/src/http/modules/ngx_http_limit_req_module.c
@@ -733,7 +733,7 @@
                                 NGX_LOG_INFO : conf->limit_log_level + 1;
 
     ngx_conf_merge_uint_value(conf->status_code, prev->status_code,
-                              NGX_HTTP_TOO_MANY_REQUESTS);
+                              NGX_HTTP_SERVICE_UNAVAILABLE);
 
     return NGX_CONF_OK;
 }