]> git.kaiwu.me - nginx.git/commit
Realip: fixed duplicate processing on redirects (ticket #1098).
authorMaxim Dounin <mdounin@mdounin.ru>
Thu, 6 Oct 2016 20:16:05 +0000 (23:16 +0300)
committerMaxim Dounin <mdounin@mdounin.ru>
Thu, 6 Oct 2016 20:16:05 +0000 (23:16 +0300)
commit52892835ef16ea15a77e2407099e0de0dfcc8a28
treebee0b2e551f969bc4874ae303efa6f91403fa92b
parent88dc647481a9280fa6cedb0bed61a34123119b3c
Realip: fixed duplicate processing on redirects (ticket #1098).

Duplicate processing was possible if the address set by realip was
listed in set_realip_from, and there was an internal redirect so module
context was cleared.  This resulted in exactly the same address being set,
so this wasn't a problem before the $realip_remote_addr variable was
introduced, though now results in incorrect $realip_remote_addr being
picked.

Fix is to use ngx_http_realip_get_module_ctx() to look up module context
even if it was cleared.  Additionally, the order of checks was switched to
check the configuration first as it looks more effective.
src/http/modules/ngx_http_realip_module.c