[ARVADOS] updated: 2.1.0-108-g5ac49e456

Git user git at public.arvados.org
Mon Nov 30 17:01:54 UTC 2020

Summary of changes:
 lib/controller/federation/conn.go | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

       via  5ac49e456c4e30a8c07a7d58fcfbfb89cac6179c (commit)
      from  a35b5abb70b41d1f3186c9639490aa4ff1af33e1 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

commit 5ac49e456c4e30a8c07a7d58fcfbfb89cac6179c
Author: Nico Cesar <nico at nicocesar.com>
Date:   Mon Nov 30 12:01:21 2020 -0500

    making old tests pass
    Arvados-DCO-1.1-Signed-off-by: Nico Cesar <nico at curii.com>

diff --git a/lib/controller/federation/conn.go b/lib/controller/federation/conn.go
index 446e3893d..7cbf4f8eb 100644
--- a/lib/controller/federation/conn.go
+++ b/lib/controller/federation/conn.go
@@ -352,7 +352,9 @@ func (conn *Conn) ContainerRequestCreate(ctx context.Context, options arvados.Cr
 			// This should probably be StatusUnauthorized
 			// (need to update test in
 			// lib/controller/federation_test.go):
-			return arvados.ContainerRequest{}, httpErrorf(http.StatusForbidden, "%w", err)
+			// When RoR is out of the picture this should be:
+			// return arvados.ContainerRequest{}, httpErrorf(http.StatusUnauthorized, "%w", err)
+			return arvados.ContainerRequest{}, httpErrorf(http.StatusForbidden, "%s", "invalid API token")
 		user, err := conn.local.UserGetCurrent(ctx, arvados.GetOptions{})
 		if err != nil {



More information about the arvados-commits mailing list