Fix panic when OIDC provider returns no name claim.

No issue #

Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom@curii.com>

diff --git a/lib/controller/localdb/login_oidc.go b/lib/controller/localdb/login_oidc.go
index 2b67a95046620c00621de88017e29124e273b5a4..74b8929a2149b14910a88823cb97a5b398a294f9 100644 (file)
--- 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)