-
Notifications
You must be signed in to change notification settings - Fork 379
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
Allow configuring a registry as http-only #2088
Comments
@mtrmac WDYT? I think the proxy misbehaves. |
I don't think the proxy necessarily misbehaves here. From its perspective, it never receives any HTTP traffic. Since it only implements HTTP, I don't think it's under any obligation to immediately terminate the connection upon receiving (what it sees as) malformed HTTP traffic. I imagine we could see similar behavior by firewalls that will silently ignore non-HTTP traffic. Therefore, I think there is still a value in having a flag that forces HTTP. I am far from a Go expert, but I've written a bit and would be willing to assist with this implementation if someone can point me in the right direction. |
As you noted, the option works as designed. I could see an argument that that design is unfortunate, but it’s no longer practical to change the semantics of the option. Adding a new option to OTOH, I also agree that fixing the proxy to recognize TLS frames when HTTP verbs are expected (like Go, in the quoted example, recognizes HTTP responses when TLS responses are expected) would probably be more generally useful than enhancing c/image — there are far fewer HTTP proxy implementations than HTTP servers and clients, so fixing the proxy would fix more of the universe. So if you are looking for a place to contribute an improvement for this end-to-end situation, I think the proxy is a place with more impact. |
From my registries.conf:
We can see that trying to pull will still try an HTTPS request first, which will fail:
Is there a way to truly get Podman to only try pulling via HTTP? I ask this for a few reasons:
I guess it isn't technically unexpected behavior, since the
registries.conf
docs say that both HTP and untrusted TLS connections will be allowed:Given the above, I'm not sure if this belongs on this issue, or should be a separate issue or feature request (e.g., implement a way to force HTTP).
Thanks!
Originally posted by @acritelli in containers/podman#16486 (comment)
The text was updated successfully, but these errors were encountered: