mirror of
https://github.com/moby/moby.git
synced 2022-11-09 12:21:53 -05:00
c06a4c22bb
This fix tries to address the issue in #22244 where the remote
API `/auth` will not set the default value of `serveraddress`
if not provided. This behavior happens after only in 1.11.0
and is a regression as in 1.10.3 `serveraddress` will be assigned
with `IndexServer` if no value is provided.
The default value `IndexServer` is assigned to `serveraddress` if
no value provided in this fix.
An integration test `TestAuthApi` has been added to cover this change
This fix fixes #22244.
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
(cherry picked from commit 93973196f4)
23 lines
676 B
Go
23 lines
676 B
Go
package main
|
|
|
|
import (
|
|
"net/http"
|
|
|
|
"github.com/docker/docker/pkg/integration/checker"
|
|
"github.com/docker/engine-api/types"
|
|
"github.com/go-check/check"
|
|
)
|
|
|
|
// Test case for #22244
|
|
func (s *DockerSuite) TestAuthApi(c *check.C) {
|
|
config := types.AuthConfig{
|
|
Username: "no-user",
|
|
Password: "no-password",
|
|
}
|
|
|
|
expected := "Get https://registry-1.docker.io/v2/: unauthorized: incorrect username or password\n"
|
|
status, body, err := sockRequest("POST", "/auth", config)
|
|
c.Assert(err, check.IsNil)
|
|
c.Assert(status, check.Equals, http.StatusUnauthorized)
|
|
c.Assert(string(body), checker.Contains, expected, check.Commentf("Expected: %v, got: %v", expected, string(body)))
|
|
}
|