----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59316/ -----------------------------------------------------------
Review request for geode, Jared Stewart, Ken Howe, Kirk Lund, and Patrick Rhomberg. Repository: geode Description ------- * using local mbs server connection will bypass all the mbean security checks * do not update the mbean attribute since pulse user has no cluster:write privilege at all * Created EmbeddedPulseRule for tests * simplify PulseAppListener * use log4j2 logging configurations Diffs ----- geode-assembly/src/test/java/org/apache/geode/test/dunit/rules/EmbeddedPulseRule.java 0ed5403d662eda1de4009d445a072ca50e4e0836 geode-assembly/src/test/java/org/apache/geode/tools/pulse/PulseSecurityTest.java 7278c843afda5ddb3b33a700c2aac56473701330 geode-assembly/src/test/java/org/apache/geode/tools/pulse/PulseSecurityWithSSLTest.java 3b9cd72d197e8a88999b83475ef14fc56f235feb geode-assembly/src/test/java/org/apache/geode/tools/pulse/PulseVerificationTest.java PRE-CREATION geode-core/src/main/java/org/apache/geode/management/internal/ManagementAgent.java 9394efb200a65b96244ef19d3ce8c6329c80652f geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/LauncherLifecycleCommands.java d61e72d9aa37a8631cfe5e5de7c593724e5fd5a0 geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/PulseAppListener.java 35f494bd11f01662b5efd01d45ff284e63d283c2 geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/controllers/PulseController.java c8788791365ac7ba5ac43228c553b1a3a48d3663 geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/data/Cluster.java 78e92d429e8f2f6c2ff370d88442cbbc1f4761a0 geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/data/IClusterUpdater.java 3ec820769a897bfb4f10a76a86dd2ce54fba3b70 geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/data/JMXDataUpdater.java c74e3cd43bc4e9d3c39c390134821472d8ace9bd geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/data/Repository.java 52e13a70f1aaa6f233b3804399b90efc3a10328b geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/security/GemFireAuthenticationProvider.java 096bf6fa39035c93b952c367c965d586df61f81e geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/security/LogoutHandler.java b169d4f6ac1c5077a48b572f0acefdd8a940309a geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/service/ClusterMemberService.java b6a129d05126c66e773ca517bc7247166f56bf05 geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/service/ClusterMembersRGraphService.java 90d4ecdb5185dc8a8e8cae5dde3b15bbc914e267 geode-pulse/src/main/resources/log4j2.properties PRE-CREATION geode-pulse/src/main/resources/pulse.properties d47d46910e46cc7a1178c835c448a0e7745b0032 geode-pulse/src/test/java/org/apache/geode/tools/pulse/controllers/PulseControllerJUnitTest.java ddd799f5158e1f4213b0b6b2de8e1853ba65ab74 Diff: https://reviews.apache.org/r/59316/diff/1/ Testing ------- Thanks, Jinmei Liao