Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@grbkdev
Copy link
Contributor

@grbkdev grbkdev commented May 17, 2025

Proposed change

Closes #4658

Type of change

  • New service widget
  • Bug fix (non-breaking change which fixes an issue)
  • New feature or enhancement (non-breaking change which adds functionality)
  • Documentation only
  • Other (please explain)

Checklist:

  • If applicable, I have added corresponding documentation changes.
  • If applicable, I have reviewed the feature / enhancement and / or service widget guidelines.
  • I have checked that all code style checks pass using pre-commit hooks and linting checks.
  • If applicable, I have tested my code for new features & regressions on both mobile & desktop devices, using the latest version of major browsers.

@shamoon
Copy link
Collaborator

shamoon commented May 17, 2025

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:

  • Docker will reject this connection
  • dockerode will attempt a TLS connection with no certs, and likely fail with a certificate required or self-signed error

Feel free to provide more context.

@shamoon shamoon closed this May 17, 2025
@grbkdev
Copy link
Contributor Author

grbkdev commented May 18, 2025

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 shamoon reopened this May 18, 2025
@shamoon shamoon changed the title Add support for specifying protocol for Docker Socket Proxy integration. Enhancement: support specifying protocol for Docker Socket Proxy integration. May 18, 2025
@shamoon shamoon changed the title Enhancement: support specifying protocol for Docker Socket Proxy integration. Enhancement: support specifying protocol for Docker Socket Proxy May 18, 2025
Copy link
Collaborator

@shamoon shamoon left a 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

@shamoon shamoon merged commit 3f160cc into gethomepage:dev May 18, 2025
4 checks passed
@grbkdev
Copy link
Contributor Author

grbkdev commented May 18, 2025

Thanks!

@github-actions
Copy link
Contributor

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.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 18, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants