You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by an...@apache.org on 2015/03/03 10:28:57 UTC
svn commit: r1663570 -
/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/SecurityProviderImpl.java
Author: angela
Date: Tue Mar 3 09:28:56 2015
New Revision: 1663570
URL: http://svn.apache.org/r1663570
Log:
minor improvement : @SuppressWarnings, simplify usage of ConfigurationParameters
Modified:
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/SecurityProviderImpl.java
Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/SecurityProviderImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/SecurityProviderImpl.java?rev=1663570&r1=1663569&r2=1663570&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/SecurityProviderImpl.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/SecurityProviderImpl.java Tue Mar 3 09:28:56 2015
@@ -207,28 +207,31 @@ public class SecurityProviderImpl implem
userAuthenticationFactory.stop();
}
+ @SuppressWarnings("UnusedDeclaration")
protected void bindPrincipalConfiguration(@Nonnull PrincipalConfiguration reference) {
principalConfiguration.addConfiguration(initConfiguration(reference));
}
+ @SuppressWarnings("UnusedDeclaration")
protected void unbindPrincipalConfiguration(@Nonnull PrincipalConfiguration reference) {
principalConfiguration.removeConfiguration(reference);
}
+ @SuppressWarnings("UnusedDeclaration")
protected void bindTokenConfiguration(@Nonnull TokenConfiguration reference) {
tokenConfiguration.addConfiguration(initConfiguration(reference));
}
+ @SuppressWarnings("UnusedDeclaration")
protected void unbindTokenConfiguration(@Nonnull TokenConfiguration reference) {
tokenConfiguration.removeConfiguration(reference);
}
//------------------------------------------------------------< private >---
private void initializeConfigurations() {
- Map<String, WhiteboardRestrictionProvider> authorizMap = ImmutableMap.of(
- AccessControlConstants.PARAM_RESTRICTION_PROVIDER, restrictionProvider
+ initConfiguration(authorizationConfiguration, ConfigurationParameters.of(
+ AccessControlConstants.PARAM_RESTRICTION_PROVIDER, restrictionProvider)
);
- initConfiguration(authorizationConfiguration, ConfigurationParameters.of(authorizMap));
Map<String, Object> userMap = ImmutableMap.<String,Object>of(
UserConstants.PARAM_AUTHORIZABLE_ACTION_PROVIDER, authorizableActionProvider,