Skip to content

registry: deprecate RepositoryInfo.Class#49006

Merged
vvoland merged 1 commit into
moby:masterfrom
thaJeztah:deprecate_repoinfo_scope
Dec 2, 2024
Merged

registry: deprecate RepositoryInfo.Class#49006
vvoland merged 1 commit into
moby:masterfrom
thaJeztah:deprecate_repoinfo_scope

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

relates to:

The Class field was added in a12b466 because Docker Hub registry required a special scope to be set for pulling plugins;

HTTP/1.1 401 Unauthorized
...
Www-Authenticate: Bearer realm="https://auth.docker.io/token",service="registry.docker.io",scope="repository(plugin):vieux/sshfs:pull",error="insufficient_scope"

This is no longer a requirement, and the field is no longer set.

This patch deprecates the field and removes its use.

- Description for the changelog

registry: deprecate `RepositoryInfo.Class`. This field is no longer used, and will be removed in the next release.

- A picture of a cute animal (not mandatory but encouraged)

The Class field was added in a12b466 because
Docker Hub registry required a special scope to be set for pulling plugins;

    HTTP/1.1 401 Unauthorized
    ...
    Www-Authenticate: Bearer realm="https://auth.docker.io/token",service="registry.docker.io",scope="repository(plugin):vieux/sshfs:pull",error="insufficient_scope"

This is no longer a requirement, and the field is no longer set.

This patch deprecates the field and removes its use.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants