milleruntime commented on a change in pull request #1798:
URL: https://github.com/apache/accumulo/pull/1798#discussion_r533738392
##########
File path: server/manager/src/main/java/org/apache/accumulo/master/Master.java
##########
@@ -1178,6 +1178,9 @@ boolean canSuspendTablets() {
log.info("All metrics modules registered");
}
+ // checking stored user hashes if any of them uses an outdated algorithm
+ security.validateStoredUserCreditentials();
Review comment:
I am wondering if this is the best place to validate all user creds. On
one hand, it may be better to do this for some situations every time Master
starts up but on the other, it may be sufficient and more efficient to only
check once on upgrade.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
