Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docker manifest fails for registries/users where JWT action "*" is restricted (e.g. gitlab) #1010

Closed
mschrupp opened this issue Apr 18, 2018 · 2 comments · Fixed by #1024
Closed

Comments

@mschrupp
Copy link

Hi everyone,

playing around with docker manifest on gitlab gives me some issues due to a bug(?) in the docker CLI.

When trying to do a docker manifest push, it fails because gitlab allows * actions only for site admins.

Now I wonder whether the action * for those commands is intentional -
shouldn't it be push and/or pull (at least that's how I understand the docs)

I think (not really sure) I tracked this problem down to the line linked below:

tokenHandler := auth.NewTokenHandler(authTransport, creds, repoName, "*")

The * action hardcoded in the last argument of the function call.

If the * action is required/intentional, I will argue in the corresponding gitlab bug instead (since that means their permission model doesn't work).

Thanks for any input on that issue.

@vdemeester
Copy link
Collaborator

cc @dmcgowan @clnperez

@clnperez
Copy link
Contributor

Thanks @vdemeester for the tag. I'll take a look.

clnperez added a commit to clnperez/cli that referenced this issue Apr 24, 2018
When creating manifest lists, don't use "*" as the permission when
creating the token handler. This causes problems with gitlab's repos.

Fixes docker#1010

Signed-off-by: Christy Norman <[email protected]>
docker-jenkins pushed a commit to docker-archive/docker-ce that referenced this issue May 2, 2018
When creating manifest lists, don't use "*" as the permission when
creating the token handler. This causes problems with gitlab's repos.

Fixes docker/cli#1010

Signed-off-by: Christy Norman <[email protected]>
Upstream-commit: c26e2264fb474f0662051ab563b3560d0b02b018
Component: cli
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 a pull request may close this issue.

3 participants