Skip to content

[27.x backport] cli/trust: GetNotaryRepository: remove uses of RepositoryInfo.Class - #5661

Merged
thaJeztah merged 1 commit into
docker:27.xfrom
thaJeztah:27.x_backport_remove_repoinfo_class
Dec 2, 2024
Merged

[27.x backport] cli/trust: GetNotaryRepository: remove uses of RepositoryInfo.Class#5661
thaJeztah merged 1 commit into
docker:27.xfrom
thaJeztah:27.x_backport_remove_repoinfo_class

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

The Class field was added 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.

updates 0ba820e

cherry picked from commit ed9fcf3)

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

The Class field was added 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.
updates 0ba820e
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
(cherry picked from commit ed9fcf3)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@codecov-commenter

codecov-commenter commented Nov 30, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.58%. Comparing base (9ea09fd) to head (5a0508c).
Report is 4 commits behind head on 27.x.

Additional details and impacted files
@@ Coverage Diff @@## 27.x #5661 +/- ##
=======================================
Coverage 58.57% 58.58% =======================================
Files 346 346 Lines 29321 29320 -1 =======================================
Hits 17176 17176 + Misses 11172 11171 -1 
Partials 973 973 

@thaJeztah
thaJeztah marked this pull request as ready for review December 2, 2024 15:15
@thaJeztah
thaJeztah merged commit b7064a2 into docker:27.xDec 2, 2024
@thaJeztah
thaJeztah deleted the 27.x_backport_remove_repoinfo_class branch December 2, 2024 15:15
Sign up for freeto 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.

4 participants

@thaJeztah@codecov-commenter@vvoland@laurazard