[arvados] updated: 2.7.0-6261-geffaaa4af8

git repository hosting git at public.arvados.org
Tue Mar 26 02:31:26 UTC 2024


Summary of changes:
 services/api/app/models/api_client_authorization.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

       via  effaaa4af8917993f250297b67d9c32a82f73197 (commit)
      from  1cbe53acb64995975bf5d951fd484451846dddc0 (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 effaaa4af8917993f250297b67d9c32a82f73197
Author: Tom Clegg <tom at curii.com>
Date:   Mon Mar 25 22:30:10 2024 -0400

    21617: Interpret any 4xx as token rejection, not an error.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom at curii.com>

diff --git a/services/api/app/models/api_client_authorization.rb b/services/api/app/models/api_client_authorization.rb
index e41fa99a08..798d49817f 100644
--- a/services/api/app/models/api_client_authorization.rb
+++ b/services/api/app/models/api_client_authorization.rb
@@ -294,8 +294,8 @@ class ApiClientAuthorization < ArvadosModel
         raise "remote cluster #{upstream_cluster_id} returned invalid token uuid #{token_uuid.inspect}"
       end
     rescue HTTPClient::BadResponseError => e
-      if e.res.status_code == 401
-        # Token is not valid.
+      if e.res.status_code >= 400 && e.res.status_code < 500
+        # Remote cluster does not accept this token.
         return nil
       end
       # CurrentApiToken#call and ApplicationController#render_error will

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list