> On May 20, 2016, 10:42 a.m., Shwetha GS wrote: > > authorizer/src/main/java/org/apache/atlas/authorize/simple/AtlasAuthorizationUtils.java, > > line 75 > > <https://reviews.apache.org/r/47612/diff/1/?file=1388279#file1388279line75> > > > > Use this syntax for all log statements. It already takes care of string > > concatenation only if the log level is enabled. if debug enabled check is > > not required: > > LOG.debug("<== AtlasAuthorizationFilter getAtlasAction HTTP Method {} > > mapped to AtlasAction : {}", method, action);
Will create another patch for this JIRA only and provide it as soon as possible. - Nixon ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47612/#review134131 ----------------------------------------------------------- On May 20, 2016, 2:02 p.m., Nixon Rodrigues wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47612/ > ----------------------------------------------------------- > > (Updated May 20, 2016, 2:02 p.m.) > > > Review request for atlas, Madhan Neethiraj, Shwetha GS, Selvamohan Neethiraj, > and Hemanth Yamijala. > > > Bugs: ATLAS-495 > https://issues.apache.org/jira/browse/ATLAS-495 > > > Repository: atlas > > > Description > ------- > > Patch contains following changes : > > 1) Refactoring of authorization code to authorizer module to be reused by > webapp, ranger plugin modules. > 1) Default policy file path to atlas-home/conf/policy-store.txt if property > is not set in atlas-application.properties file. > 2) Renamed create action instead of write. > 4) Added authorizer impl class in atlas properties and fall back to simple > authorizer if authorizer is not set in property. > 5) AtlasAuthorizerFactory for authorizer. > 6) Handled changes in Test cases and in filter as per new implementation of > AtlasAuthorizerFactory. > > > Diffs > ----- > > authorization/pom.xml PRE-CREATION > > authorization/src/main/java/org/apache/atlas/authorize/AtlasAccessRequest.java > PRE-CREATION > > authorization/src/main/java/org/apache/atlas/authorize/AtlasActionTypes.java > PRE-CREATION > > authorization/src/main/java/org/apache/atlas/authorize/AtlasAuthorizationException.java > PRE-CREATION > authorization/src/main/java/org/apache/atlas/authorize/AtlasAuthorizer.java > PRE-CREATION > > authorization/src/main/java/org/apache/atlas/authorize/AtlasAuthorizerFactory.java > PRE-CREATION > > authorization/src/main/java/org/apache/atlas/authorize/AtlasResourceTypes.java > PRE-CREATION > > authorization/src/main/java/org/apache/atlas/authorize/simple/AtlasAuthorizationUtils.java > PRE-CREATION > > authorization/src/main/java/org/apache/atlas/authorize/simple/FileReaderUtil.java > PRE-CREATION > > authorization/src/main/java/org/apache/atlas/authorize/simple/PolicyDef.java > PRE-CREATION > > authorization/src/main/java/org/apache/atlas/authorize/simple/PolicyParser.java > PRE-CREATION > > authorization/src/main/java/org/apache/atlas/authorize/simple/PolicyUtil.java > PRE-CREATION > > authorization/src/main/java/org/apache/atlas/authorize/simple/SimpleAtlasAuthorizer.java > PRE-CREATION > > authorization/src/test/java/org/apache/atlas/authorize/simple/AtlasAuthorizationUtilsTest.java > PRE-CREATION > > authorization/src/test/java/org/apache/atlas/authorize/simple/PolicyParserTest.java > PRE-CREATION > > authorization/src/test/java/org/apache/atlas/authorize/simple/PolicyUtilTest.java > PRE-CREATION > > authorization/src/test/java/org/apache/atlas/authorize/simple/SimpleAtlasAuthorizerTest.java > PRE-CREATION > common/pom.xml 614b3f6 > common/src/main/java/org/apache/atlas/utils/PropertiesUtil.java > PRE-CREATION > common/src/main/java/org/apache/atlas/utils/XMLPropertiesUtil.java > PRE-CREATION > distro/src/conf/atlas-application.properties e2082ef > distro/src/conf/policy-store.txt 5bcc1cf > distro/src/conf/users-credentials.properties 212d018 > pom.xml 685eb7d > webapp/pom.xml 4b67ffa > webapp/src/main/java/org/apache/atlas/authorize/AtlasAccessRequest.java > 5db9646 > webapp/src/main/java/org/apache/atlas/authorize/AtlasAccessorTypes.java > 5f3827a > webapp/src/main/java/org/apache/atlas/authorize/AtlasActionTypes.java > 13c8b53 > > webapp/src/main/java/org/apache/atlas/authorize/AtlasAuthorizationException.java > 676c9f9 > > webapp/src/main/java/org/apache/atlas/authorize/AtlasAuthorizationUtils.java > 14a2aac > webapp/src/main/java/org/apache/atlas/authorize/AtlasAuthorizer.java > 7c93c7a > webapp/src/main/java/org/apache/atlas/authorize/AtlasResourceTypes.java > 14a72f1 > webapp/src/main/java/org/apache/atlas/authorize/PolicyDef.java 0ee39df > webapp/src/main/java/org/apache/atlas/authorize/PolicyParser.java 51a6dc2 > webapp/src/main/java/org/apache/atlas/authorize/PolicyUtil.java a565f96 > webapp/src/main/java/org/apache/atlas/authorize/SimpleAtlasAuthorizer.java > 2a32e4e > webapp/src/main/java/org/apache/atlas/util/FileReaderUtil.java 22eaff9 > webapp/src/main/java/org/apache/atlas/util/PropertiesUtil.java fef8efb > webapp/src/main/java/org/apache/atlas/util/XMLPropertiesUtil.java 9c4f1c7 > > webapp/src/main/java/org/apache/atlas/web/filters/AtlasAuthorizationFilter.java > 13fc7da > > webapp/src/main/java/org/apache/atlas/web/security/AtlasADAuthenticationProvider.java > 9e5df45 > > webapp/src/main/java/org/apache/atlas/web/security/AtlasLdapAuthenticationProvider.java > e66b930 > webapp/src/main/webapp/WEB-INF/applicationContext.xml b58952c > > webapp/src/test/java/org/apache/atlas/authorize/AtlasAuthorizationUtilsTest.java > 5fc4420 > webapp/src/test/java/org/apache/atlas/authorize/PolicyParserTest.java > 507d4c6 > webapp/src/test/java/org/apache/atlas/authorize/PolicyUtilTest.java 59e88c9 > > webapp/src/test/java/org/apache/atlas/authorize/SimpleAtlasAuthorizerTest.java > 5041e6f > > Diff: https://reviews.apache.org/r/47612/diff/ > > > Testing > ------- > > Verified :: Rat check and mvn clean install > > Manually verified policy enforcement for 3 different groups. > > > Thanks, > > Nixon Rodrigues > >