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

Support Cloudflare? #20

Closed
fluxsauce opened this issue Mar 2, 2017 · 3 comments
Closed

Support Cloudflare? #20

fluxsauce opened this issue Mar 2, 2017 · 3 comments

Comments

@fluxsauce
Copy link
Contributor

https://support.cloudflare.com/hc/en-us/articles/200170986-How-does-Cloudflare-handle-HTTP-Request-headers-

First exception: CF-Connecting-IP

To provide the client (visitor) IP address for every request to the origin, Cloudflare adds the CF-Connecting-IP header.

"CF-Connecting-IP: A.B.C.D"

where A.B.C.D is the client's IP address, also known as the original visitor IP address.

and

Sixth Exception: True-Client-IP Enterprise Plan only

To provide the client (visitor) IP address for every request to the origin, Cloudflare adds the True-Client-IP header.

"True-Client-IP: A.B.C.D"

where A.B.C.D is the client's IP address, also known as the original visitor IP address. This request header is only available on our Enterprise plan.

If you're willing, I can submit a PR.

@pbojinov
Copy link
Owner

pbojinov commented Mar 2, 2017

PR would be wonderful! Thank you

@fluxsauce
Copy link
Contributor Author

fluxsauce commented Mar 2, 2017

Cool, can do. Is there anything blocking #19 ? If not, I'd rather build on that one to avoid a potential merge conflict.

@pbojinov
Copy link
Owner

pbojinov commented Mar 3, 2017

Nothing blocking #19. Feel free to build off that.

I'll go ahead and merge that into master after I finish running some 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

No branches or pull requests

2 participants