Browse Source

Merge pull request #445 from docker/acr_login_cleanup

Use existing oauth Token struct rather than create our own
Guillaume Tardif 5 years ago
parent
commit
3c7e551940
4 changed files with 10 additions and 8 deletions
  1. 2 1
      aci/backend.go
  2. 3 2
      aci/convert/convert_test.go
  3. 3 4
      aci/convert/registry_credentials.go
  4. 2 1
      example/backend.go

+ 2 - 1
aci/backend.go

@@ -30,6 +30,8 @@ import (
 	"github.com/pkg/errors"
 	"github.com/pkg/errors"
 	"github.com/sirupsen/logrus"
 	"github.com/sirupsen/logrus"
 
 
+	ecstypes "github.com/docker/ecs-plugin/pkg/compose"
+
 	"github.com/docker/api/aci/convert"
 	"github.com/docker/api/aci/convert"
 	"github.com/docker/api/aci/login"
 	"github.com/docker/api/aci/login"
 	"github.com/docker/api/backend"
 	"github.com/docker/api/backend"
@@ -39,7 +41,6 @@ import (
 	"github.com/docker/api/context/cloud"
 	"github.com/docker/api/context/cloud"
 	"github.com/docker/api/context/store"
 	"github.com/docker/api/context/store"
 	"github.com/docker/api/errdefs"
 	"github.com/docker/api/errdefs"
-	ecstypes "github.com/docker/ecs-plugin/pkg/compose"
 )
 )
 
 
 const (
 const (

+ 3 - 2
aci/convert/convert_test.go

@@ -23,10 +23,11 @@ import (
 	"github.com/Azure/azure-sdk-for-go/profiles/latest/containerinstance/mgmt/containerinstance"
 	"github.com/Azure/azure-sdk-for-go/profiles/latest/containerinstance/mgmt/containerinstance"
 	"github.com/Azure/go-autorest/autorest/to"
 	"github.com/Azure/go-autorest/autorest/to"
 	"github.com/compose-spec/compose-go/types"
 	"github.com/compose-spec/compose-go/types"
-	"github.com/docker/api/containers"
-	"github.com/docker/api/context/store"
 	"gotest.tools/v3/assert"
 	"gotest.tools/v3/assert"
 	is "gotest.tools/v3/assert/cmp"
 	is "gotest.tools/v3/assert/cmp"
+
+	"github.com/docker/api/containers"
+	"github.com/docker/api/context/store"
 )
 )
 
 
 var convertCtx = store.AciContext{
 var convertCtx = store.AciContext{

+ 3 - 4
aci/convert/registry_credentials.go

@@ -26,6 +26,8 @@ import (
 	"os/exec"
 	"os/exec"
 	"strings"
 	"strings"
 
 
+	"golang.org/x/oauth2"
+
 	"github.com/Azure/azure-sdk-for-go/profiles/latest/containerinstance/mgmt/containerinstance"
 	"github.com/Azure/azure-sdk-for-go/profiles/latest/containerinstance/mgmt/containerinstance"
 	"github.com/Azure/go-autorest/autorest/to"
 	"github.com/Azure/go-autorest/autorest/to"
 	compose "github.com/compose-spec/compose-go/types"
 	compose "github.com/compose-spec/compose-go/types"
@@ -167,10 +169,7 @@ func (c cliRegistryHelper) autoLoginAcr(registry string) error {
 		return err
 		return err
 	}
 	}
 
 
-	type acrToken struct {
-		RefreshToken string `json:"refresh_token"`
-	}
-	newToken := acrToken{}
+	newToken := oauth2.Token{}
 	if err := json.Unmarshal(bits, &newToken); err != nil {
 	if err := json.Unmarshal(bits, &newToken); err != nil {
 		return err
 		return err
 	}
 	}

+ 2 - 1
example/backend.go

@@ -28,10 +28,11 @@ import (
 	"github.com/docker/api/context/cloud"
 	"github.com/docker/api/context/cloud"
 	"github.com/docker/api/errdefs"
 	"github.com/docker/api/errdefs"
 
 
+	ecstypes "github.com/docker/ecs-plugin/pkg/compose"
+
 	"github.com/docker/api/backend"
 	"github.com/docker/api/backend"
 	"github.com/docker/api/compose"
 	"github.com/docker/api/compose"
 	"github.com/docker/api/containers"
 	"github.com/docker/api/containers"
-	ecstypes "github.com/docker/ecs-plugin/pkg/compose"
 )
 )
 
 
 type apiService struct {
 type apiService struct {