isapir commented on issue #157: https://bz.apache.org/bugzilla/show_bug.cgi?id=63333 URL: https://github.com/apache/tomcat/pull/157#issuecomment-481861275 > Ok, let me know what you think we should put as a comment additionally that is so greatly missed and it is not obvious enough for you or someone else and I will add it > because if you to have another thread trying to access the LockOutRealm and you put the 'invocationSuccess = true' on line 405 you could just change the failed invocation e.g. invocationSuccess from false to true on line 405 to only put the variable back to false when the next attempt fails again Well then, here's an excellent opportunity for a comment so that someone wouldn't edit it later without taking this into account.
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org