id summary reporter owner description type status priority milestone component version resolution keywords cc uname nginx_version 2340 decoded percent-encoded values written to logfiles mozai@… "Saw that bad requests that percent-encode nonprintable characters are breaking the nginx log files. currently it breaks logfile parsers, but I worry it could be used to inject out-of-band instructions into logfiles. Seen: In /var/log/nginx/error.log: 2022/03/27 17:04:52 [error] 23762#23762: *1142443 ""/srv/http/com.dandytype/ /example.com/index.html"" is not found (2: No such file or directory), client: 194.195.86.50, server: dandytype.com, request: ""GET /%0A/example.com/ HTTP/1.1"", host: ""chorus.mozai.com"" What I expected: Logfile has one record per newline-terminated string What I saw instead: Logfile has two records for one event, neither matching the pattern for logfile records. Other information: I worry that writing decoded percent-encoded bytes could be used to insert nulls (%00) that could be used for buffer underruns in C, or insert Unicode codepoints to obfuscate requests by bad actors." defect closed minor nginx-core 1.14.x duplicate logging "Linux kanaya.mozai.com 5.14.17-x86_64-linode150 #1 SMP Thu Nov 11 13:17:05 EST 2021 x86_64 x86_64 x86_64 GNU/Linux " "nginx version: nginx/1.14.0 (Ubuntu) built with OpenSSL 1.1.1 11 Sep 2018 TLS SNI support enabled configure arguments: --with-cc-opt='-g -O2 -fdebug-prefix-map=/build/nginx-H4cN7P/nginx-1.14.0=. -fstack-protector-strong -Wformat -Werror=format-security -fPIC -Wdate-time -D_FORTIFY_SOURCE=2' --with-ld-opt='-Wl,-Bsymbolic-functions -Wl,-z,relro -Wl,-z,now -fPIC' --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 --modules-path=/usr/lib/nginx/modules --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-http_ssl_module --with-http_stub_status_module --with-http_realip_module --with-http_auth_request_module --with-http_v2_module --with-http_dav_module --with-http_slice_module --with-threads --with-http_addition_module --with-http_geoip_module=dynamic --with-http_gunzip_module --with-http_gzip_static_module --with-http_image_filter_module=dynamic --with-http_sub_module --with-http_xslt_module=dynamic --with-stream=dynamic --with-stream_ssl_module --with-mail=dynamic --with-mail_ssl_module "