17014: rebase with master and adaptation changes

author Nico Cesar <nico@nicocesar.com>

Mon, 16 Nov 2020 19:43:38 +0000 (14:43 -0500)

committer Nico Cesar <nico@nicocesar.com>

Mon, 16 Nov 2020 19:43:38 +0000 (14:43 -0500)
author Nico Cesar <nico@nicocesar.com>
Mon, 16 Nov 2020 19:43:38 +0000 (14:43 -0500)
committer Nico Cesar <nico@nicocesar.com>
Mon, 16 Nov 2020 19:43:38 +0000 (14:43 -0500)
diff --git a/lib/controller/localdb/login_oidc.go b/lib/controller/localdb/login_oidc.go

index a41a93da1d1c440463ad5180749539ac6e326830..b99a1c2aa5b9fc47f4d461615873cc6204c24b80 100644 (file)
--- a/lib/controller/localdb/login_oidc.go
+++ b/lib/controller/localdb/login_oidc.go
@@ -22,7 +22,6 @@ import (
         "time"
  
         "git.arvados.org/arvados.git/lib/controller/api"
-       "git.arvados.org/arvados.git/lib/controller/railsproxy"
         "git.arvados.org/arvados.git/lib/controller/rpc"
         "git.arvados.org/arvados.git/lib/ctrlctx"
         "git.arvados.org/arvados.git/sdk/go/arvados"
@@ -322,7 +321,7 @@ func OIDCAccessTokenAuthorizer(cluster *arvados.Cluster, getdb func(context.Cont
         // We want ctrl to be nil if the chosen controller is not a
         // *oidcLoginController, so we can ignore the 2nd return value
         // of this type cast.
-       ctrl, _ := chooseLoginController(cluster, railsproxy.NewConn(cluster)).(*oidcLoginController)
+       ctrl, _ := NewConn(cluster).loginController.(*oidcLoginController)
         cache, err := lru.New2Q(tokenCacheSize)
         if err != nil {
                 panic(err)
@@ -474,7 +473,7 @@ func (ta *oidcTokenAuthorizer) registerToken(ctx context.Context, tok string) er
                 }
                 ctxlog.FromContext(ctx).WithField("HMAC", hmac).Debug("(*oidcTokenAuthorizer)registerToken: updated api_client_authorizations row")
         } else {
-               aca, err = createAPIClientAuthorization(ctx, ta.ctrl.RailsProxy, ta.ctrl.Cluster.SystemRootToken, *authinfo)
+               aca, err = ta.ctrl.Parent.CreateAPIClientAuthorization(ctx, ta.ctrl.Cluster.SystemRootToken, *authinfo)
                 if err != nil {
                         return err
                 }
diff --git a/sdk/go/arvadostest/api.go b/sdk/go/arvadostest/api.go

index fe600257f7cdc38485612b033bbd15ff04e105ab..df3e46febd5598f9d065fba24c964701b480c1d1 100644 (file)
--- a/sdk/go/arvadostest/api.go
+++ b/sdk/go/arvadostest/api.go
@@ -106,23 +106,23 @@ func (as *APIStub) ContainerUnlock(ctx context.Context, options arvados.GetOptio
         return arvados.Container{}, as.Error
  }
  func (as *APIStub) ContainerRequestCreate(ctx context.Context, options arvados.CreateOptions) (arvados.ContainerRequest, error) {
-       as.appendCall(as.ContainerRequestCreate, ctx, options)
+       as.appendCall(ctx, as.ContainerRequestCreate, options)
         return arvados.ContainerRequest{}, as.Error
  }
  func (as *APIStub) ContainerRequestUpdate(ctx context.Context, options arvados.UpdateOptions) (arvados.ContainerRequest, error) {
-       as.appendCall(as.ContainerRequestUpdate, ctx, options)
+       as.appendCall(ctx, as.ContainerRequestUpdate, options)
         return arvados.ContainerRequest{}, as.Error
  }
  func (as *APIStub) ContainerRequestGet(ctx context.Context, options arvados.GetOptions) (arvados.ContainerRequest, error) {
-       as.appendCall(as.ContainerRequestGet, ctx, options)
+       as.appendCall(ctx, as.ContainerRequestGet, options)
         return arvados.ContainerRequest{}, as.Error
  }
  func (as *APIStub) ContainerRequestList(ctx context.Context, options arvados.ListOptions) (arvados.ContainerRequestList, error) {
-       as.appendCall(as.ContainerRequestList, ctx, options)
+       as.appendCall(ctx, as.ContainerRequestList, options)
         return arvados.ContainerRequestList{}, as.Error
  }
  func (as *APIStub) ContainerRequestDelete(ctx context.Context, options arvados.DeleteOptions) (arvados.ContainerRequest, error) {
-       as.appendCall(as.ContainerRequestDelete, ctx, options)
+       as.appendCall(ctx, as.ContainerRequestDelete, options)
         return arvados.ContainerRequest{}, as.Error
  }
  func (as *APIStub) SpecimenCreate(ctx context.Context, options arvados.CreateOptions) (arvados.Specimen, error) {
author	Nico Cesar <nico@nicocesar.com>
	Mon, 16 Nov 2020 19:43:38 +0000 (14:43 -0500)
committer	Nico Cesar <nico@nicocesar.com>
	Mon, 16 Nov 2020 19:43:38 +0000 (14:43 -0500)
lib/controller/localdb/login_oidc.go		patch \| blob \| history
sdk/go/arvadostest/api.go		patch \| blob \| history