[ARVADOS] updated: 2.1.0-85-gf56945838

Git user git at public.arvados.org
Mon Feb 15 04:02:48 UTC 2021


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

       via  f569458386b8e8c1b9d910762abae9bd9ca9506d (commit)
      from  e65d692c62b5ef4963905114d096027059838573 (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 f569458386b8e8c1b9d910762abae9bd9ca9506d
Author: Tom Clegg <tom at curii.com>
Date:   Sun Feb 14 23:00:40 2021 -0500

    17335: Fix panic when OIDC provider returns no name claim.
    
    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 688499d9e..93b9145a4 100644
--- a/lib/controller/localdb/login_oidc.go
+++ b/lib/controller/localdb/login_oidc.go
@@ -156,7 +156,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