accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <>
Subject [GitHub] [accumulo] ctubbsii edited a comment on issue #1221: New Security Interface
Date Thu, 20 Jun 2019 00:39:46 GMT
ctubbsii edited a comment on issue #1221: New Security Interface
   The `SecurityOperation` class is currently an internal only class. If this interface were
to represent something like we have today, it should look something more like:
   public interface Security {
     void initialize(SiteConfiguration configuration);
     Authenticator authenticator();
     Authorizor authorizor();
     PolicyChecker policyChecker(); // or PermissionsHandler
   I don't think we need the second layer interface, `SecurityOperation` if we're going to
re-design the pluggable security.
   Also, if this does become proper SPI, it should use a more generic configuration type (`Properties`?),
rather than expose our internal `SiteConfiguration` configuration class.

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:

With regards,
Apache Git Services

View raw message