Skip to content
Snippets Groups Projects
Commit 84181c4b authored by Patryk Kazimierowski's avatar Patryk Kazimierowski
Browse files

fixed condition

parent 47a2760d
Branches
Tags
2 merge requests!139fixed condition,!137Draft: Release/1.7.1
Pipeline #93263 passed
...@@ -49,31 +49,23 @@ public class OidcUserServiceImpl implements OidcUserService { ...@@ -49,31 +49,23 @@ public class OidcUserServiceImpl implements OidcUserService {
boolean existUserBySamlToken = userService boolean existUserBySamlToken = userService
.existsBySamlToken(oidcUserSub); .existsBySamlToken(oidcUserSub);
boolean existUserByUsernameAsSamlToken = userService
.existsBySamlToken(oidcUserPreferredUsername);
boolean existsUserBySamlTokenAsEmail = userService
.existsBySamlToken(oidcUserEmail);
if (existUserBySamlToken) { if (existUserBySamlToken) {
return userService return userService
.findBySamlToken(oidcUserSub) .findBySamlToken(oidcUserSub)
.orElseThrow(); .orElseThrow();
} else if (existUserByUsernameAsSamlToken || existsUserBySamlTokenAsEmail) { }
User user = userService if(userService.existsByEmail(oidcUserEmail)) {
.findBySamlToken(oidcUserPreferredUsername) User user = userService.findByEmail(oidcUserEmail);
.orElseThrow(); if (user.getSamlToken().equals(oidcUserEmail)
if (user.getEmail().equals(oidcUserEmail)) { || user.getSamlToken().equals(oidcUserPreferredUsername)) {
user.setSamlToken(oidcUserSub); user.setSamlToken(oidcUserSub);
userService.update(user); userService.update(user);
return user; return user;
} else { }
throw new ExternalUserMatchException("External user "
+ oidcUserPreferredUsername
+ " does not match internal user ");
}
} else {
return registerNewUser(oidcUser);
} }
return registerNewUser(oidcUser);
} }
@Override @Override
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment