-
Notifications
You must be signed in to change notification settings - Fork 125
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
Documentation suggestion: haproxy #9
Comments
Cool - It looks like I could use some explanation as well. It sounds like HAProxy uses a different x-forwared-for header than expected? Does that code define which header to use ( Thanks! |
That's right. This arguably belongs in the Laravel docs, but since your entire module is a patch for the Laravel proxy system, I figure this is the right place for a patch to the proxy docs as well. |
Now I'm second-guessing myself on this one. I'm beginning to believe that my whole X-forwarded-scheme vs X-forwarded-proto weirdness was the result of following someone's nginx + haproxy recipe blindly. Never mind. |
Let me know what you find! It's totally possible haproxy does something On Monday, March 31, 2014, dsadinoff notifications@github.com wrote:
|
I've also had this problem when using proxypass in Apache with AWS load balancer and Laravel doesn't know the original request was https so the asset urls use http. Is it possible that line you added in the beginning could fix this? |
Marking as duplicate to #7, as the solution is the same. I'm hoping to add that as a configuration available (and documenting it). Thanks! |
For proxies such as haproxy, I would suggest that we mention the following snippet in the docs:
The text was updated successfully, but these errors were encountered: