17335: Fix panic when OIDC provider returns no name claim.
authorTom Clegg <tom@curii.com>
Mon, 15 Feb 2021 04:00:40 +0000 (23:00 -0500)
committerTom Clegg <tom@curii.com>
Mon, 15 Feb 2021 04:02:43 +0000 (23:02 -0500)
Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom@curii.com>

lib/controller/localdb/login_oidc.go

index 688499d9e54a7b7d92f0741cbeade0aa82fc638d..93b9145a4529a4f7dde365e382959c177121d168 100644 (file)
@@ -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)