Skip to content

Commit 1f7e5a0

Browse files
RANGER-3825: Ranger admin user is unable to change another user email after the upgrade
1 parent c72e2e3 commit 1f7e5a0

File tree

1 file changed

+8
-8
lines changed
  • security-admin/src/main/java/org/apache/ranger/biz

1 file changed

+8
-8
lines changed

security-admin/src/main/java/org/apache/ranger/biz/UserMgr.java

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -530,17 +530,17 @@ public VXPortalUser changeEmailAddress(XXPortalUser gjUser,
530530
MessageEnums.OPER_NO_PERMISSION, null, null, ""
531531
+ changeEmail);
532532
}
533-
} else {
534-
String encryptedOldPwd = encrypt(gjUser.getLoginId(), changeEmail.getOldPassword());
533+
} else {
534+
String encryptedOldPwd = encrypt(gjUser.getLoginId(), changeEmail.getOldPassword());
535+
if (!stringUtil.equals(encryptedOldPwd, gjUser.getPassword())) {
536+
encryptedOldPwd = encryptWithOlderAlgo(gjUser.getLoginId(), changeEmail.getOldPassword());
535537
if (!stringUtil.equals(encryptedOldPwd, gjUser.getPassword())) {
536-
logger.info("changeEmailAddress(). Invalid password. changeEmail="
537-
+ changeEmail);
538-
throw restErrorUtil.createRESTException(
539-
"serverMsg.userMgrWrongPassword",
540-
MessageEnums.OPER_NO_PERMISSION, null, null, ""
541-
+ changeEmail);
538+
logger.info("changeEmailAddress(). Invalid password. changeEmail=" + changeEmail);
539+
throw restErrorUtil.createRESTException("serverMsg.userMgrWrongPassword",
540+
MessageEnums.OPER_NO_PERMISSION, null, null, "" + changeEmail);
542541
}
543542
}
543+
}
544544

545545
// Normalize email. Make it lower case
546546
gjUser.setEmailAddress(stringUtil.normalizeEmail(changeEmail

0 commit comments

Comments
 (0)