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

Utilise X-FORWARDED-[FOR|PORT] headers -- Means things work properly when behind a reverse proxy. #397

Closed
wants to merge 2 commits into from

Conversation

DanBUK
Copy link

@DanBUK DanBUK commented Jul 19, 2011

Quite a simple patch really; but necessary when working behind reverse proxies.

@tedeh
Copy link

tedeh commented Jul 20, 2011

+1, would save me a lot of headache

@DanBUK
Copy link
Author

DanBUK commented Jul 20, 2011

I just fixed this pull request. I was referencing the wrong object before.

@rauchg
Copy link
Contributor

rauchg commented Jul 31, 2011

x-forwarded-for can be a comma separated list of values, depending on the hops the client made:

http://en.wikipedia.org/wiki/X-Forwarded-For#Format

Please fix your patch and add tests =]

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

Successfully merging this pull request may close these issues.

3 participants