Merge pull request #55 from wadahiro/rpeper-conditional-discriminator

fix: eliminate the possibility of NullPointerException
wadahiro · Nov 2, 2024 · ac7d0e3 · ac7d0e3
2 parents 5c0a0d7 + b8a84df
commit ac7d0e3
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/main/java/org/keycloak/social/discord/DiscordIdentityProvider.java b/src/main/java/org/keycloak/social/discord/DiscordIdentityProvider.java
@@ -71,9 +71,11 @@ protected BrokeredIdentityContext extractIdentityFromProfile(EventBuilder event,
 
         String username = getJsonProperty(profile, "username");
         String discriminator = getJsonProperty(profile, "discriminator");
-        if (!discriminator.equals("0")) {
-            username = username + "#" + discriminator;
+
+        if (!"0".equals(discriminator)) {
+            username += "#" + discriminator;
         }
+
         user.setUsername(username);
         user.setEmail(getJsonProperty(profile, "email"));
         user.setIdp(this);