diff options
| author | Matej Vasek <mvasek@redhat.com> | 2021-09-15 19:12:31 +0200 |
|---|---|---|
| committer | Matej Vasek <mvasek@redhat.com> | 2021-09-15 20:01:02 +0200 |
| commit | 7c5d64b4783c2bf05587b3a4672b35c1e32066c7 (patch) | |
| tree | 35d8e645c0bf2b4c10b2c2f0d79db894ea6a51a5 | |
| parent | c0cde378298bd8d5183155c8f62aae574be069eb (diff) | |
| download | podman-7c5d64b4783c2bf05587b3a4672b35c1e32066c7.tar.gz podman-7c5d64b4783c2bf05587b3a4672b35c1e32066c7.tar.bz2 podman-7c5d64b4783c2bf05587b3a4672b35c1e32066c7.zip | |
Fix /auth compat endpoint
Signed-off-by: Matej Vasek <mvasek@redhat.com>
| -rw-r--r-- | pkg/api/handlers/compat/auth.go | 16 | ||||
| -rw-r--r-- | test/apiv2/60-auth.at | 9 |
2 files changed, 20 insertions, 5 deletions
diff --git a/pkg/api/handlers/compat/auth.go b/pkg/api/handlers/compat/auth.go index 2244adc3d..cf53e060c 100644 --- a/pkg/api/handlers/compat/auth.go +++ b/pkg/api/handlers/compat/auth.go @@ -50,9 +50,19 @@ func Auth(w http.ResponseWriter, r *http.Request) { Status: "Login Succeeded", }) } else { - utils.WriteResponse(w, http.StatusBadRequest, entities.AuthReport{ - IdentityToken: "", - Status: "login attempt to " + authConfig.ServerAddress + " failed with status: " + err.Error(), + var msg string + + var unauthErr DockerClient.ErrUnauthorizedForCredentials + if errors.As(err, &unauthErr) { + msg = "401 Unauthorized" + } else { + msg = err.Error() + } + + utils.WriteResponse(w, http.StatusInternalServerError, struct { + Message string `json:"message"` + }{ + Message: "login attempt to " + authConfig.ServerAddress + " failed with status: " + msg, }) } } diff --git a/test/apiv2/60-auth.at b/test/apiv2/60-auth.at index cfde519c1..1e087d12b 100644 --- a/test/apiv2/60-auth.at +++ b/test/apiv2/60-auth.at @@ -5,10 +5,15 @@ start_registry +# Test unreachable +t POST /v1.40/auth username=$REGISTRY_USERNAME password=WrOnGPassWord serveraddress=does.not.exist.io:1234/ \ + 500 \ + .message~'.*no such host.*' + # Test with wrong password. Confirm bad status and appropriate error message t POST /v1.40/auth username=$REGISTRY_USERNAME password=WrOnGPassWord serveraddress=localhost:$REGISTRY_PORT/ \ - 400 \ - .Status~'.* invalid username/password' + 500 \ + .message~'.* 401 Unauthorized' # Test with the right password. Confirm status message t POST /v1.40/auth username=$REGISTRY_USERNAME password=$REGISTRY_PASSWORD serveraddress=localhost:$REGISTRY_PORT/ \ |