docker / docker/docker-py

docker.images.push does not throw an exception on authentication failures

Open
#3,277 0 comments 3 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
7.2k
Forks
1.7k
Avg merge
13d 8h
Merged PRs (30d)
2

Description

when calling images.push, I would expect an exception if the push fails for any reason. Instead a list of strings containing json(!) is returned.

In my case this was caused by using a credentials that were able to read the repository, but not to write on it.
I am using docker-py 7.1.0.

>>> d.images.push('registry.devops.company.de/sdm/devenv/docker/nds22:DMT')
'{"status":"The push refers to repository [registry.devops.company.de/sdm/devenv/docker/nds22]"}\r\n{"status":"Preparing","progressDetail":{},"id":"e13fc22196f7"}\r\n{"status":"Preparing","progressDetail":{},"id":"d3f568e44044"}\r\n{"status":"Preparing","progressDetail":{},"id":"20d52c42c0d2"}\r\n{"status":"Preparing","progressDetail":{},"id":"3b970ba98564"}\r\n{"status":"Preparing","progressDetail":{},"id":"a3fe5767be1f"}\r\n{"status":"Preparing","progressDetail":{},"id":"dc056a2a3bd7"}\r\n{"status":"Preparing","progressDetail":{},"id":"94edc0573192"}\r\n{"status":"Preparing","progressDetail":{},"id":"190bd049e69a"}\r\n{"status":"Preparing","progressDetail":{},"id":"c1278f9a1d1a"}\r\n{"status":"Preparing","progressDetail":{},"id":"c1278f9a1d1a"}\r\n{"status":"Preparing","progressDetail":{},"id":"284b4e718772"}\r\n{"status":"Preparing","progressDetail":{},"id":"17f90e156a5c"}\r\n{"status":"Preparing","progressDetail":{},"id":"0c0d9af5e66d"}\r\n{"status":"Preparing","progressDetail":{},"id":"b1809d9a4ad6"}\r\n{"status":"Preparing","progressDetail":{},"id":"d24d1c4d4104"}\r\n{"status":"Preparing","progressDetail":{},"id":"06e6efc8f7ad"}\r\n{"status":"Waiting","progressDetail":{},"id":"dc056a2a3bd7"}\r\n{"status":"Waiting","progressDetail":{},"id":"94edc0573192"}\r\n{"status":"Preparing","progressDetail":{},"id":"79a69a8c6a35"}\r\n{"status":"Waiting","progressDetail":{},"id":"190bd049e69a"}\r\n{"status":"Preparing","progressDetail":{},"id":"6fb0da284158"}\r\n{"status":"Waiting","progressDetail":{},"id":"c1278f9a1d1a"}\r\n{"status":"Preparing","progressDetail":{},"id":"0d9950f614d6"}\r\n{"status":"Preparing","progressDetail":{},"id":"fb793371e751"}\r\n{"status":"Preparing","progressDetail":{},"id":"22f8be0594d4"}\r\n{"status":"Preparing","progressDetail":{},"id":"b6cfde0b8194"}\r\n{"status":"Preparing","progressDetail":{},"id":"fc39326d9866"}\r\n{"status":"Preparing","progressDetail":{},"id":"f3779f3ebb98"}\r\n{"status":"Preparing","progressDetail":{},"id":"e524eb3d960d"}\r\n{"status":"Preparing","progressDetail":{},"id":"33bf517fef8b"}\r\n{"status":"Waiting","progressDetail":{},"id":"284b4e718772"}\r\n{"status":"Preparing","progressDetail":{},"id":"eeb727f8e4bb"}\r\n{"status":"Waiting","progressDetail":{},"id":"17f90e156a5c"}\r\n{"status":"Waiting","progressDetail":{},"id":"0c0d9af5e66d"}\r\n{"status":"Preparing","progressDetail":{},"id":"e6ebdd044b9c"}\r\n{"status":"Waiting","progressDetail":{},"id":"b1809d9a4ad6"}\r\n{"status":"Waiting","progressDetail":{},"id":"d24d1c4d4104"}\r\n{"status":"Preparing","progressDetail":{},"id":"5817476c92af"}\r\n{"status":"Waiting","progressDetail":{},"id":"06e6efc8f7ad"}\r\n{"status":"Preparing","progressDetail":{},"id":"06590ea36d6c"}\r\n{"status":"Waiting","progressDetail":{},"id":"79a69a8c6a35"}\r\n{"status":"Preparing","progressDetail":{},"id":"508a97bc2a5f"}\r\n{"status":"Waiting","progressDetail":{},"id":"6fb0da284158"}\r\n{"status":"Waiting","progressDetail":{},"id":"22f8be0594d4"}\r\n{"status":"Waiting","progressDetail":{},"id":"f3779f3ebb98"}\r\n{"status":"Preparing","progressDetail":{},"id":"80f4edff0ff6"}\r\n{"status":"Waiting","progressDetail":{},"id":"fc39326d9866"}\r\n{"status":"Waiting","progressDetail":{},"id":"0d9950f614d6"}\r\n{"status":"Waiting","progressDetail":{},"id":"b6cfde0b8194"}\r\n{"status":"Waiting","progressDetail":{},"id":"fb793371e751"}\r\n{"status":"Waiting","progressDetail":{},"id":"e524eb3d960d"}\r\n{"status":"Waiting","progressDetail":{},"id":"5817476c92af"}\r\n{"status":"Waiting","progressDetail":{},"id":"e6ebdd044b9c"}\r\n{"status":"Preparing","progressDetail":{},"id":"be3202997979"}\r\n{"status":"Waiting","progressDetail":{},"id":"33bf517fef8b"}\r\n{"status":"Waiting","progressDetail":{},"id":"eeb727f8e4bb"}\r\n{"status":"Waiting","progressDetail":{},"id":"06590ea36d6c"}\r\n{"status":"Preparing","progressDetail":{},"id":"638f2a99f7f3"}\r\n{"status":"Waiting","progressDetail":{},"id":"508a97bc2a5f"}\r\n{"status":"Waiting","progressDetail":{},"id":"80f4edff0ff6"}\r\n{"status":"Preparing","progressDetail":{},"id":"1f6ca1411ece"}\r\n{"status":"Waiting","progressDetail":{},"id":"be3202997979"}\r\n{"status":"Preparing","progressDetail":{},"id":"5efb24c93836"}\r\n{"status":"Waiting","progressDetail":{},"id":"1f6ca1411ece"}\r\n{"status":"Preparing","progressDetail":{},"id":"c6fb32907cb8"}\r\n{"status":"Waiting","progressDetail":{},"id":"5efb24c93836"}\r\n{"status":"Preparing","progressDetail":{},"id":"9d92b36b6f8d"}\r\n{"status":"Waiting","progressDetail":{},"id":"c6fb32907cb8"}\r\n{"status":"Preparing","progressDetail":{},"id":"7bce3075df58"}\r\n{"status":"Waiting","progressDetail":{},"id":"9d92b36b6f8d"}\r\n{"status":"Preparing","progressDetail":{},"id":"bcf30f7550b0"}\r\n{"status":"Waiting","progressDetail":{},"id":"7bce3075df58"}\r\n{"status":"Preparing","progressDetail":{},"id":"74ddd0ec08fa"}\r\n{"status":"Waiting","progressDetail":{},"id":"bcf30f7550b0"}\r\n{"status":"Waiting","progressDetail":{},"id":"74ddd0ec08fa"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"20d52c42c0d2"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"e13fc22196f7"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"3b970ba98564"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"d3f568e44044"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"a3fe5767be1f"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"dc056a2a3bd7"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"c1278f9a1d1a"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"284b4e718772"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"190bd049e69a"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"94edc0573192"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"17f90e156a5c"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"0c0d9af5e66d"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"06e6efc8f7ad"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"d24d1c4d4104"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"b1809d9a4ad6"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"79a69a8c6a35"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"0d9950f614d6"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"6fb0da284158"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"22f8be0594d4"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"fb793371e751"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"b6cfde0b8194"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"fc39326d9866"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"f3779f3ebb98"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"e524eb3d960d"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"33bf517fef8b"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"eeb727f8e4bb"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"e6ebdd044b9c"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"5817476c92af"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"06590ea36d6c"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"508a97bc2a5f"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"80f4edff0ff6"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"1f6ca1411ece"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"be3202997979"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"638f2a99f7f3"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"5efb24c93836"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"c6fb32907cb8"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"9d92b36b6f8d"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"7bce3075df58"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"bcf30f7550b0"}\r\n{"status":"Layer already exists","progressDetail":{},"id":"74ddd0ec08fa"}\r\n{"errorDetail":{"message":"errors:\\ndenied: requested access to the resource is denied\\nunauthorized: authentication required\\n"},"error":"errors:\\ndenied: requested access to the resource is denied\\nunauthorized: authentication required\\n"}\r\n'

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start at the docker-py implementation of images.push and trace how its JSON response is handled, using the shown authentication-failure response as the reproduction. Add coverage for denied push credentials and verify that the call raises an exception rather than returning the response.

Written by the indexing model from the issue text.

Assessment

Tech stack
docker, python
Domain
api, backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.