----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51810/ -----------------------------------------------------------
Review request for geode, Kevin Duling, Kirk Lund, and Swapnil Bawaskar. Repository: geode Description ------- GEODE-17: pass the properties created by authInit to the security manager Our GeodeSecurityUtil will always only takes a username and password, while the client/peer's authInit geneates a Properties object which the SecurityManager will eventually use to authenticate. This change will make sure the Properties generated by the authInits will be sent intact to the SecurityManager. Still in the case of gfsh/pulse/jmx login, we would send in the username/password in the Properties with the key of "security-username" and "security-password". So any implementation of SecurityManager needs to handle these special keys. Diffs ----- geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/auth/GMSAuthenticator.java 7fc212cf5af54c128644aa6a63579654b79a58e4 geode-core/src/main/java/com/gemstone/gemfire/internal/cache/tier/sockets/HandShake.java 8dfb476b4238399790d9520ae902b65bfc71c60f geode-core/src/main/java/com/gemstone/gemfire/internal/security/GeodeSecurityUtil.java 5fa5564831faadce7efd338a74b09172009d1b99 geode-core/src/main/java/com/gemstone/gemfire/internal/security/IntegratedSecurityService.java a6ec7fa146881c3fed20adfc45db9ebed0bbf0a1 geode-core/src/main/java/com/gemstone/gemfire/internal/security/SecurityService.java dd8c69b894a3e244111dcd2425fcd9d425356a1a geode-core/src/main/java/com/gemstone/gemfire/internal/security/shiro/CustomAuthRealm.java 09b8d5f4e1062546bbdd19dfe0ba6124590022a8 geode-core/src/main/java/com/gemstone/gemfire/internal/security/shiro/GeodeAuthenticationToken.java PRE-CREATION geode-core/src/main/java/com/gemstone/gemfire/internal/security/shiro/JMXShiroAuthenticator.java fe895d4b4ef2f95aee774f1b191831423fd72037 geode-core/src/main/java/com/gemstone/gemfire/management/internal/web/controllers/support/LoginHandlerInterceptor.java c831d0a99f99a12e8af29716e7a4c3ee4c75069f geode-core/src/test/java/com/gemstone/gemfire/distributed/internal/membership/gms/auth/GMSAuthenticatorWithSecurityManagerTest.java b00b3ff01ae5c718f45ced6ce2106705ad24f12e geode-core/src/test/java/com/gemstone/gemfire/security/IntegratedClientAuthDUnitTest.java 0f53cbc220c61158c992938d4ea108c24d7a9969 Diff: https://reviews.apache.org/r/51810/diff/ Testing ------- precheckin running Thanks, Jinmei Liao