fix: github idp - test length of the array before accessing the secon…

…d element fixes gravitee-io/issues#5507
gravitee-io · Aug 3, 2021 · 783b45b · 783b45b
1 parent 61e9fb8
commit 783b45b
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/...o/gravitee/rest/api/model/configuration/identity/github/GitHubIdentityProviderEntity.java b/...o/gravitee/rest/api/model/configuration/identity/github/GitHubIdentityProviderEntity.java
@@ -87,8 +87,11 @@ public Map<String, String> getUserProfileMapping() {
                 put(UserProfile.ID, isEmailRequired() ? "email" : "id");
                 put(UserProfile.SUB, "id");
                 put(UserProfile.EMAIL, "email");
-                put(UserProfile.FIRSTNAME, "{#jsonPath(#profile, '$.name').split(' ')[1]}");
-                put(UserProfile.LASTNAME, "{#jsonPath(#profile, '$.name').split(' ')[0]}");
+                put(UserProfile.FIRSTNAME, "{#jsonPath(#profile, '$.name').split(' ')[0]}");
+                put(
+                    UserProfile.LASTNAME,
+                    "{#jsonPath(#profile, '$.name').split(' ').length == 1 ? #jsonPath(#profile, '$.name').split(' ')[0] : #jsonPath(#profile, '$.name').split(' ')[1]}"
+                );
                 put(UserProfile.PICTURE, "avatar_url");
             }
         };