[ARVADOS] updated: 2.1.0-98-g10f8671a0

Git user git at public.arvados.org
Mon Nov 16 19:59:30 UTC 2020


Summary of changes:
 lib/controller/federation.go | 5 -----
 1 file changed, 5 deletions(-)

       via  10f8671a06406ea09915de693a0ab578505072bb (commit)
      from  00ca5332212f0e7f091d9e952c07e2205b255ec6 (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 10f8671a06406ea09915de693a0ab578505072bb
Author: Nico Cesar <nico at nicocesar.com>
Date:   Mon Nov 16 14:58:41 2020 -0500

    following https://dev.arvados.org/issues/17014#note-8
    
    removed "len(token) < 41" case
    
    Arvados-DCO-1.1-Signed-off-by: Nico Cesar <nico at curii.com>

diff --git a/lib/controller/federation.go b/lib/controller/federation.go
index 869fec58e..bc93cc9ea 100644
--- a/lib/controller/federation.go
+++ b/lib/controller/federation.go
@@ -165,11 +165,6 @@ func (h *Handler) validateAPItoken(req *http.Request, token string) (*CurrentUse
 		token = sp[2]
 	}
 
-	if len(token) < 41 {
-		ctxlog.FromContext(req.Context()).Debugf("validateAPItoken(%s): The lenght of the token is not 41", token)
-		return nil, false, nil
-	}
-
 	user.Authorization.APIToken = token
 	var scopes string
 	err = db.QueryRowContext(req.Context(), `SELECT api_client_authorizations.uuid, api_client_authorizations.scopes, users.uuid FROM api_client_authorizations JOIN users on api_client_authorizations.user_id=users.id WHERE api_token=$1 AND (expires_at IS NULL OR expires_at > current_timestamp AT TIME ZONE 'UTC') LIMIT 1`, token).Scan(&user.Authorization.UUID, &scopes, &user.UUID)

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list