Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#943 closed defect (invalid)

Nginx transforms HTTP method from POST to GET

Reported by: svenmueller@… Owned by:
Priority: minor Milestone:
Component: nginx-core Version: 1.4.x
Keywords: method transform access log Cc: ops@…, dev-sphere@…
uname -a: Linux node-gloomy-lichterman.sphere-ci.cloud.commercetools.de 3.13.0-79-generic #123-Ubuntu SMP Fri Feb 19 14:27:58 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
nginx -V: nginx version: nginx/1.4.6 (Ubuntu)
built by gcc 4.8.2 (Ubuntu 4.8.2-19ubuntu1)
TLS SNI support enabled
configure arguments: --with-cc-opt='-g -O2 -fstack-protector --param=ssp-buffer-size=4 -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2' --with-ld-opt='-Wl,-Bsymbolic-functions -Wl,-z,relro' --prefix=/usr/share/nginx --conf-path=/etc/nginx/nginx.conf --http-log-path=/var/log/nginx/access.log --error-log-path=/var/log/nginx/error.log --lock-path=/var/lock/nginx.lock --pid-path=/run/nginx.pid --http-client-body-temp-path=/var/lib/nginx/body --http-fastcgi-temp-path=/var/lib/nginx/fastcgi --http-proxy-temp-path=/var/lib/nginx/proxy --http-scgi-temp-path=/var/lib/nginx/scgi --http-uwsgi-temp-path=/var/lib/nginx/uwsgi --with-debug --with-pcre-jit --with-ipv6 --with-http_ssl_module --with-http_stub_status_module --with-http_realip_module --with-http_addition_module --with-http_dav_module --with-http_flv_module --with-http_geoip_module --with-http_gzip_static_module --with-http_image_filter_module --with-http_mp4_module --with-http_perl_module --with-http_random_index_module --with-http_secure_link_module --with-http_spdy_module --with-http_sub_module --with-http_xslt_module --with-mail --with-mail_ssl_module --add-module=/build/nginx-BU0ZJt/nginx-1.4.6/debian/modules/headers-more-nginx-module --add-module=/build/nginx-BU0ZJt/nginx-1.4.6/debian/modules/nginx-auth-pam --add-module=/build/nginx-BU0ZJt/nginx-1.4.6/debian/modules/nginx-cache-purge --add-module=/build/nginx-BU0ZJt/nginx-1.4.6/debian/modules/nginx-dav-ext-module --add-module=/build/nginx-BU0ZJt/nginx-1.4.6/debian/modules/nginx-development-kit --add-module=/build/nginx-BU0ZJt/nginx-1.4.6/debian/modules/nginx-echo --add-module=/build/nginx-BU0ZJt/nginx-1.4.6/debian/modules/ngx-fancyindex --add-module=/build/nginx-BU0ZJt/nginx-1.4.6/debian/modules/nginx-http-push --add-module=/build/nginx-BU0ZJt/nginx-1.4.6/debian/modules/nginx-lua --add-module=/build/nginx-BU0ZJt/nginx-1.4.6/debian/modules/nginx-upload-progress --add-module=/build/nginx-BU0ZJt/nginx-1.4.6/debian/modules/nginx-upstream-fair --add-module=/build/nginx-BU0ZJt/nginx-1.4.6/debian/modules/ngx_http_substitutions_filter_module

Description

The following POST request contains a duplicate HEADER value to trigger a Nginx header parsing error:

curl -X POST -v -H "Authorization: 1" -H "Authorization: 2" http://localhost/

After the request was processed by Nginx, the log shows HTTP method "GET" instead of "POST"

192.168.99.1 - - [30/Mar/2016:15:16:53 +0000] "GET / HTTP/1.1" 400 173 "-" "curl/7.43.0" "-"

Expected HTTP method to be logged is "POST".

Change History (2)

comment:1 by Maxim Dounin, 8 years ago

Resolution: invalid
Status: newclosed

The $request as logged to the access_log is a stringnginx got from client. If you see "GET / HTTP/1.1" there - it means that nginx got "GET", not "POST".

I suspect the problem in your configuration, which likely uses error_page to handle 400 errors, and then proxies them somewhere else. The error_page changes the request method to GET unless you use named location.

Please use mailing list if you need further help.

comment:2 by svenmueller@…, 8 years ago

Yes, we are using directive error_page. Could you please give an example how to use custom error pages with named locations so that the GET method is not changed? I tried it mayself but without success so far.

Thx a lot,
Sven

Note: See TracTickets for help on using tickets.