accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
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
URL: https://github.com/apache/accumulo/issues/1221#issuecomment-503795758
 
 
   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:
   
   ```java
   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:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message