-
Notifications
You must be signed in to change notification settings - Fork 78
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
IPv6 client IP not updated when using KeepAlive #37
Comments
I already did that; see my second paragraph. By switching from stable to master, 71a17f2 is included in my source and fixed the problem for IPv4 IPs. It did not fix the problem for IPv6 IPs, thus this issue. |
Oh, sorry, missed that! |
Explanation and proposed fix in #45. :) |
This issue may have been fixed with @BenSjoberg patch which has now been merged, can you please confirm if you are still experiencing the issue still? |
Would this fix have any impact on #42 ? |
Interestingly, on current master this problem reappeared for me. On new connections:
Second request on same connection:
Turning off keep-alives helped. |
Currently using the "stable" tree we were facing the problem that only the first request has the real client IP correctly transferred into REMOTE_ADDR when using KeepAlive. Further requests received the unchanged proxy IP, until the KeepAlive timeout value has been reached.
After stumbling upon Issue #35 we changed from "stable" to the current "master" tree, which effectively fixed the problem - but, amazingly, only for IPv4 addresses: IPv6 addresses still only got correctly set on every first request ever or after reaching the KeepAlive timeout. We're testing with the following PHP script:
Result on the first request is (correctly):
Result on every next request within the KeepAlive timeout results in (185.26.156.43 is the proxy IP listed as RPAF_ProxyIPs):
As a workaround we're currently having to turn off KeepAlive; in this case mod_rpaf provides correct results for both IPv4 and IPv6. Is there any chance to fix this?
The text was updated successfully, but these errors were encountered: