<div dir="auto">This should be a major release patch. It's breaking for everyone passing the error log.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 8 Jul 2021, 7:17 pm Jeremie Drouet, <<a href="mailto:jeremie.drouet@gmail.com">jeremie.drouet@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"># HG changeset patch<br>
# User Jeremie Drouet <<a href="mailto:jeremie.drouet@gmail.com" target="_blank" rel="noreferrer">jeremie.drouet@gmail.com</a>><br>
# Date 1625150632 -7200<br>
# Thu Jul 01 16:43:52 2021 +0200<br>
# Node ID 7db380334d2ca671b98ab7563bab9ddee501c573<br>
# Parent e0fdd75871e40e4c57ed405cadb94235550142c9<br>
Rename referrer to referer in error log<br>
<br>
According to the developer doc from mozilla, referrer should be<br>
written with only one R and everywhere in the nginx codebase, we<br>
are using referer with only one R, so to make it consistent, I<br>
propose you to replace referrer with two R in the error log<br>
message with referer with only one R.<br>
I'm also proposing this because of the work I'm doing on vector<br>
regarding a nginx log parser in which we end up having referer<br>
and referrer when we parse logs.<br>
<br>
<a href="https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Referer" rel="noreferrer noreferrer" target="_blank">https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Referer</a><br>
<a href="https://github.com/timberio/vector/" rel="noreferrer noreferrer" target="_blank">https://github.com/timberio/vector/</a><br>
<br>
diff -r e0fdd75871e4 -r 7db380334d2c src/http/ngx_http_request.c<br>
--- a/src/http/ngx_http_request.c Mon Jun 28 18:01:24 2021 +0300<br>
+++ b/src/http/ngx_http_request.c Thu Jul 01 16:43:52 2021 +0200<br>
@@ -3889,7 +3889,7 @@<br>
}<br>
<br>
if (r->headers_in.referer) {<br>
- p = ngx_snprintf(buf, len, ", referrer: \"%V\"",<br>
+ p = ngx_snprintf(buf, len, ", referer: \"%V\"",<br>
&r->headers_in.referer->value);<br>
buf = p;<br>
}<br>
<br>
_______________________________________________<br>
nginx-devel mailing list<br>
<a href="mailto:nginx-devel@nginx.org" target="_blank" rel="noreferrer">nginx-devel@nginx.org</a><br>
<a href="http://mailman.nginx.org/mailman/listinfo/nginx-devel" rel="noreferrer noreferrer" target="_blank">http://mailman.nginx.org/mailman/listinfo/nginx-devel</a><br>
</blockquote></div>