-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Enhancement: support specifying protocol for Docker Socket Proxy #5285
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
Conversation
|
Thanks, but I dont believe this does what you think it does. Docker daemons only speak HTTPS when they're configured with TLS, and the client must provide matching certs — protocol: "https" on its own won't work. Sure, someone can specify https, but this bypasses our existing (correct) TLS guardrails, so in practice:
Feel free to provide more context. |
|
Please look at the linked discussion. I am running Docker Socket Proxy behind reverse proxy with HTTPS. Without this change Homepage tries to make an API call to DSP using dockerode over HTTP and fails. I understand this is not a typical scenario, but it is a valid scenario, and dockerode supports it through the "https" parameter that I am just "pulling up" into Homepage configuration. I tested it locally. |
shamoon
left a comment
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.
I think this will confuse folks but the change is fine. Tried to make the docs clearer
|
Thanks! |
|
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion for related concerns. See our contributing guidelines for more details. |
Proposed change
Closes #4658
Type of change
Checklist: