----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52067/ -----------------------------------------------------------
(Updated Sept. 19, 2016, 9:29 p.m.) Review request for geode, Jared Stewart, Kevin Duling, and Kirk Lund. Repository: geode Description ------- GEODE-1909: add authorization in GMSAuthenticator The user who get authenticated in the cluster needs to have CLUSTER:MANAGE previlage as well. Note this is NOT the user who issued "start server" command in gfsh, but is the peer user whose credential is put in the server's security.properties file. Diffs (updated) ----- geode-core/src/main/java/org/apache/geode/distributed/internal/membership/gms/GMSMemberFactory.java 08178e92ad9b4db30534c80873c63e8b4c11808f geode-core/src/main/java/org/apache/geode/distributed/internal/membership/gms/auth/GMSAuthenticator.java 3f030c9af4ab0503ad201c9172fe61a256eb961a geode-core/src/main/java/org/apache/geode/distributed/internal/membership/gms/membership/GMSJoinLeave.java d15e8bfa09b3c24d8c2db10d524bfce0d4b61ad5 geode-core/src/main/java/org/apache/geode/internal/i18n/LocalizedStrings.java b61201ef60b4dc3ea6d42e7ae30d305e2ebb7e35 geode-core/src/test/java/org/apache/geode/distributed/internal/membership/gms/auth/GMSAuthenticatorWithAuthenticatorTest.java 45a5881207179e950bb40a897660f63f583a4e19 geode-core/src/test/java/org/apache/geode/distributed/internal/membership/gms/auth/GMSAuthenticatorWithSecurityManagerTest.java ddb208d043a8503b52119e363ff23c54acd0a4f4 geode-core/src/test/java/org/apache/geode/security/StartServerAuthorizationTest.java PRE-CREATION geode-core/src/test/resources/org/apache/geode/management/internal/security/cacheServer.json 3bb3e2f010da9e97af3ade60dbd4b4b45b9370b3 Diff: https://reviews.apache.org/r/52067/diff/ Testing ------- precheckin running Thanks, Jinmei Liao