Skip to content

Comments

dockerconfig: add fallback for loading DHI and Scout credentials#3612

Merged
tonistiigi merged 1 commit intodocker:masterfrom
tonistiigi:dhi-auth
Jan 16, 2026
Merged

dockerconfig: add fallback for loading DHI and Scout credentials#3612
tonistiigi merged 1 commit intodocker:masterfrom
tonistiigi:dhi-auth

Conversation

@tonistiigi
Copy link
Member

These are also Docker-managed registries with the same users.

Fall back to the main hostname if not logged in to the exact domain.

Note that this does not currently work with imagetools as the auth system there has not yet been combined with new config loader from #3562 .

@crazy-max @cdupuis

@tonistiigi tonistiigi added this to the v0.31.0 milestone Jan 16, 2026
crazy-max
crazy-max previously approved these changes Jan 16, 2026
These are also Docker managed registries with same users.

Fall back to main hostname if not logged in to exact domain.

Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
@tonistiigi tonistiigi merged commit 9c33ef3 into docker:master Jan 16, 2026
159 checks passed
)

const (
dockerHubRegistryHost = "registry-1.docker.io"
Copy link

@mathieu-benoit mathieu-benoit Jan 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @tonistiigi @crazy-max @cdupuis, I just tried this new feature by using the associated released version v0.31.0 in the docker/setup-buildx-action. Out of curiosity, why it's registry-1.docker.io and not docker.io here?
Context: the default value for the registry in docker/login-action is docker.io.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's the actual hostname for main hub registry. https://docs.docker.com/reference/api/registry/latest/#tag/pull docker.io is redirect to docker.com if you send requests directly to that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants