-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52067/
-----------------------------------------------------------

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
-----

  
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

Reply via email to