Skip to content

Conversation

@thaJeztah
Copy link
Member

Now that 3f5b1bd removed DCT, which needed some of the intermediate types (indexInfo), we can simplify the auth code further and just get the base64-encoded AuthConfig to be set as header.

- What I did

- How I did it

- How to verify it

- Human readable description for the release notes

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

Now that 3f5b1bd removed DCT, which
needed some of the intermediate types (indexInfo), we can simplify the
auth code further and just get the base64-encoded AuthConfig to be set
as header.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah added this to the 29.0.0 milestone Aug 26, 2025
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
cli/command/plugin/push.go 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@thaJeztah thaJeztah requested a review from Benehiko August 26, 2025 14:53
@thaJeztah thaJeztah merged commit 6bc7ed8 into docker:master Aug 26, 2025
113 of 115 checks passed
@thaJeztah thaJeztah deleted the plugin_simplify_auth branch August 26, 2025 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants