DonalEvans commented on a change in pull request #7116:
URL: https://github.com/apache/geode/pull/7116#discussion_r753571471
##########
File path:
geode-core/src/main/java/org/apache/geode/internal/security/IntegratedSecurityService.java
##########
@@ -119,8 +121,12 @@ public Subject getSubject() {
}
}
- // in other cases like rest call, client operations, we get it from the
current thread
- currentUser = SecurityUtils.getSubject();
+ try {
+ // in other cases like rest call, client operations, we get it from the
current thread
+ currentUser = SecurityUtils.getSubject();
+ } catch (UnavailableSecurityManagerException e) {
+ throw new CacheClosedException("likely cache shutdown in progress", e);
Review comment:
Is it possible to check the value of `GemfireCahceImpl.isClosing` here
to confirm if we are actually closing the cache, rather than just assuming that
we are?
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
