Skip to content
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

request->address can return multiple IP addresses #1235

Open
ichesnokov opened this issue Jun 20, 2022 · 0 comments
Open

request->address can return multiple IP addresses #1235

ichesnokov opened this issue Jun 20, 2022 · 0 comments

Comments

@ichesnokov
Copy link
Contributor

When behind_proxy is on, request->address returns value of X-Forwarded-For header, and it's completely legitimate for this header to contain multiple IP addresses: https://developer.mozilla.org/ru/docs/Web/HTTP/Headers/X-Forwarded-For - IP of the client and proxies' IPs.

While according to Dancer documentation it should only Return the IP address of the client.

It would be nice to make it only return client's IP - either first IP from the list, or $ENV{REMOTE_ADDR} (like e.g. Dancer2 does).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant