Skip to content

Update docker resolver to authorize redirects#5504

Merged
estesp merged 1 commit intocontainerd:masterfrom
dmcgowan:fix-authorization-on-redirect
May 17, 2021
Merged

Update docker resolver to authorize redirects#5504
estesp merged 1 commit intocontainerd:masterfrom
dmcgowan:fix-authorization-on-redirect

Conversation

@dmcgowan
Copy link
Copy Markdown
Member

Allows redirects to be authorized if authorization is provided for the redirected to host. The authorization will always go to the redirect and never to the referrer.

Matches redirect logic done by Docker today.

Fixes #5503

Allows redirects to be authorized if authorization is provided
for the redirected to host. The authorization will always go
to the redirect and never to the referrer.

Signed-off-by: Derek McGowan <derek@mcg.dev>
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented May 14, 2021

Build succeeded.

Copy link
Copy Markdown
Member

@samuelkarp samuelkarp left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM

@estesp estesp merged commit bf1a667 into containerd:master May 17, 2021
@thaJeztah thaJeztah added cherry-picked/1.4.x PR commits are cherry picked into the release/1.4 branch cherry-picked/1.5.x PR commits are cherry-picked into release/1.5 branch and removed cherry-pick/1.5.x labels Jul 20, 2021
breakings added a commit to breakings/packages that referenced this pull request Aug 8, 2021
The ninth patch release for containerd 1.4 updates runc to 1.0.1 and contains
other minor updates.

### Notable Updates

- **Update runc binary to 1.0.1** [#5751](containerd/containerd#5751)
- **Update pull authorization logic on redirect** [#5504](containerd/containerd#5504)
- **Fix user agent used for fetching registry authentication tokens** [#5761](containerd/containerd#5761)
@dmcgowan dmcgowan deleted the fix-authorization-on-redirect branch April 20, 2024 00:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-picked/1.4.x PR commits are cherry picked into the release/1.4 branch cherry-picked/1.5.x PR commits are cherry-picked into release/1.5 branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unable to pull image from public registry nabo.codimd.dev

7 participants