<div dir="ltr">Hi Maxim,<div><br></div><div>Thanks for you comment! Do you have any other approaches/suggestions?</div><div>I use nginx as a proxy/load-balancer. The request will be processed by the upstream java servers. I assume my change won't actually modify the actual header, so upstream will still get the original header and can distinguish . and - ? </div><div><br></div><div>Regards,</div><div>Frank</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 17, 2015 at 4:31 AM, Maxim Dounin <span dir="ltr"><<a href="mailto:mdounin@mdounin.ru" target="_blank">mdounin@mdounin.ru</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello!<br>
<span class=""><br>
On Sat, Aug 15, 2015 at 12:15:47AM -0700, Frank Liu wrote:<br>
<br>
> I made the below patch and can now use $upstream_http_x_header for<br>
> logformat to capture the header X.header in the access log. Does anybody<br>
> see any issues with the patch?<br>
><br>
> --- src/http/ngx_http_variables.c.orig 2015-08-15 02:19:31.635328112 +0000<br>
><br>
> +++ src/http/ngx_http_variables.c 2015-08-15 02:19:42.051541422 +0000<br>
><br>
> @@ -897,6 +897,8 @@<br>
><br>
><br>
><br>
>              } else if (ch == '-') {<br>
><br>
>                  ch = '_';<br>
><br>
> +            } else if (ch == '.') {<br>
><br>
> +                ch = '_';<br>
><br>
>              }<br>
<br>
</span>Such approach will likely result in security problems, as<br>
"X.header" and "X-header" would be indistinguishable from nginx<br>
point of view.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Maxim Dounin<br>
<a href="http://nginx.org/" rel="noreferrer" target="_blank">http://nginx.org/</a><br>
</font></span><div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
nginx mailing list<br>
<a href="mailto:nginx@nginx.org">nginx@nginx.org</a><br>
<a href="http://mailman.nginx.org/mailman/listinfo/nginx" rel="noreferrer" target="_blank">http://mailman.nginx.org/mailman/listinfo/nginx</a><br>
</div></div></blockquote></div><br></div>