UserService activateRegistration method didn't set activation key properly
kevintanhongann opened this issue · comments
Overview of the issue
Reproduce the error
- Go through register
- Go through activation
- Login will be unsuccessful.
Related issues
N/A
Suggest a Fix
public Optional<User> activateRegistration(String key) {
log.debug("Activating user for activation key {}", key);
return userRepository
.findOneByActivationKey(key)
.map(user -> {
// activate given user for the registration key.
user.setActivated(true);
user.setActivationKey(null);
this.clearUserCaches(user);
log.debug("Activated user: {}", user);
return user;
});
}
to
public Optional<User> activateRegistration(String key) {
log.debug("Activating user for activation key {}", key);
return userRepository
.findOneByActivationKey(key)
.map(user -> {
// activate given user for the registration key.
user.setActivated(true);
user.setActivationKey(key); //the key should be set here.
this.clearUserCaches(user);
log.debug("Activated user: {}", user);
return user;
});
}
Project configuration
Any project that doesn't use third party auth.
Entity configuration(s) entityName.json
files generated in the .jhipster
directory
N/A
Browsers and Operating System
Any
- Checking this box is mandatory (this is just to show you read everything)