[ARVADOS] updated: 2.1.0-446-g5dc622e37

Git user git at public.arvados.org
Mon Feb 15 04:05:17 UTC 2021


Summary of changes:
 lib/controller/localdb/login_oidc.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

       via  5dc622e37805f511b04eb66557441b28fba13b80 (commit)
      from  4dc6c2c37fba1c59e3e57ad47a1ee55d0593ed04 (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 5dc622e37805f511b04eb66557441b28fba13b80
Author: Tom Clegg <tom at curii.com>
Date:   Sun Feb 14 23:04:13 2021 -0500

    Fix panic when OIDC provider returns no name claim.
    
    No issue #
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom at curii.com>

diff --git a/lib/controller/localdb/login_oidc.go b/lib/controller/localdb/login_oidc.go
index 2b67a9504..74b8929a2 100644
--- a/lib/controller/localdb/login_oidc.go
+++ b/lib/controller/localdb/login_oidc.go
@@ -176,7 +176,7 @@ func (ctrl *oidcLoginController) getAuthInfo(ctx context.Context, token *oauth2.
 		if names := strings.Fields(strings.TrimSpace(name)); len(names) > 1 {
 			ret.FirstName = strings.Join(names[0:len(names)-1], " ")
 			ret.LastName = names[len(names)-1]
-		} else {
+		} else if len(names) > 0 {
 			ret.FirstName = names[0]
 		}
 		ret.Email, _ = claims[ctrl.EmailClaim].(string)

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list