-
Notifications
You must be signed in to change notification settings - Fork 17
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
Add trusted proxies to docs #1086
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add some information regarding docker? Because with the www container it is a bit special isnt it?
This is a complex question indeed. Since we are talking about Docker, I believe that the We already have I'm personally a bit confused whether we might even need set Then read also the defaults section: https://caddyserver.com/docs/caddyfile/directives/reverse_proxy#defaults. I must say I'm not running the Docker setup at the moment, so I'm unable to validate these configs in a real-world scenario to be fully honest. Finally, we also have this rework in place, which might change everything again: #1085 |
So maybe a follow up issue then? |
That would be easier yes. |
Or we just add to use |
but then again, it doesn't fit the Nginx section here. And in the other PR we are also changing the docs for Docker. |
See title.