jmelchio commented on a change in pull request #7116:
URL: https://github.com/apache/geode/pull/7116#discussion_r753449577
##########
File path:
geode-core/src/main/java/org/apache/geode/internal/cache/GemFireCacheImpl.java
##########
@@ -1099,6 +1099,30 @@ public void lockDiskStore(String diskStoreName) {
doLockDiskStore(diskStoreName);
}
+ /**
+ * Having this method to return a SecurityServiceFactory allows users to
replace the factory
+ * for testing purposes.
+ *
+ * In a test set the system property
"org.apache.geode.internal.security.SecurityServiceFactory"
+ * to the class of choice ie. [Classname].class.getName() and in this class
either implement
+ * SecurityServiceFactory or override the default implementation to create
the desired behavior.
Review comment:
@DonalEvans I agree with that but unfortunately the code is such that I
have not been able to come up with an alternative approach.
--
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]
