----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51227/#review146223 -----------------------------------------------------------
geode-core/src/main/java/com/gemstone/gemfire/distributed/ConfigurationProperties.java (line 158) <https://reviews.apache.org/r/51227/#comment212592> There are a number of deprecated SSL properties in this file that say to use other properties, but those properties are also deprecated an point to the new properties you've added. In order to avoid confusion they should all point to your new properties. See SERVER_SSL_CIPHERS, for instance. ReviewBoard has really made a mess of your diffs. There are also a large number of formatting changes to wade through. Can you do a walkthrough for us? - Bruce Schuchardt On Aug. 18, 2016, 11:30 p.m., Udo Kohlmeyer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51227/ > ----------------------------------------------------------- > > (Updated Aug. 18, 2016, 11:30 p.m.) > > > Review request for geode, Bruce Schuchardt and Hitesh Khamesra. > > > Repository: geode > > > Description > ------- > > As per the specification: > https://cwiki.apache.org/confluence/display/GEODE/Revised+SSL+properties > > * Removal of legacy/deprecate previous ssl-* properties > * Deprecation of current <component>-ssl-* properties > * Moving of SocketCreator and SocketCloser to the > com.gemstone.gemfire.internal.net package > * Addition of an ExtendedAliasKeyManager to manage keystores with many keys > and aliases > * Addition of locator ssl properties > * Addition of SocketCreatorFactory > * Addition of SSLConfigFactory > * Addition of JMXMBeanDUnitTest > * Addition of SSLEnabledComponent Enum > * Ciphers and Protocols are stored as String[] instead of previous legacy > String > * TCPClient is now instance rather than Singleton > > > Diffs > ----- > > > geode-assembly/src/test/java/com/gemstone/gemfire/management/internal/configuration/SharedConfigurationEndToEndDUnitTest.java > 3408717 > > geode-assembly/src/test/java/com/gemstone/gemfire/rest/internal/web/controllers/RestAPIsQueryAndFEJUnitTest.java > 8321caf > > geode-assembly/src/test/java/com/gemstone/gemfire/rest/internal/web/controllers/RestAPIsWithSSLDUnitTest.java > daa781d > > geode-core/src/main/java/com/gemstone/gemfire/admin/DistributedSystemConfig.java > 5ef389f > > geode-core/src/main/java/com/gemstone/gemfire/admin/GemFireMemberStatus.java > 1b544a8 > > geode-core/src/main/java/com/gemstone/gemfire/admin/internal/DistributedSystemConfigImpl.java > 517f5a6 > > geode-core/src/main/java/com/gemstone/gemfire/admin/internal/DistributedSystemHealthMonitor.java > 54e7de7 > > geode-core/src/main/java/com/gemstone/gemfire/admin/internal/DistributionLocatorConfigImpl.java > 88d939a > > geode-core/src/main/java/com/gemstone/gemfire/admin/internal/EnabledManagedEntityController.java > ff139c5 > > geode-core/src/main/java/com/gemstone/gemfire/admin/internal/InetAddressUtil.java > 43d8e44 > > geode-core/src/main/java/com/gemstone/gemfire/admin/internal/ManagedEntityConfigImpl.java > 69751e9 > > geode-core/src/main/java/com/gemstone/gemfire/admin/jmx/internal/AgentConfigImpl.java > a71c479 > > geode-core/src/main/java/com/gemstone/gemfire/admin/jmx/internal/AgentLauncher.java > dfa9ac3 > > geode-core/src/main/java/com/gemstone/gemfire/admin/jmx/internal/MX4JServerSocketFactory.java > 649038e > > geode-core/src/main/java/com/gemstone/gemfire/cache/client/internal/AutoConnectionSourceImpl.java > 844a775 > > geode-core/src/main/java/com/gemstone/gemfire/cache/client/internal/ConnectionFactoryImpl.java > 56449db > > geode-core/src/main/java/com/gemstone/gemfire/cache/client/internal/ConnectionImpl.java > 6ed2912 > > geode-core/src/main/java/com/gemstone/gemfire/distributed/ConfigurationProperties.java > 5c3a282 > geode-core/src/main/java/com/gemstone/gemfire/distributed/Locator.java > 8795b71 > > geode-core/src/main/java/com/gemstone/gemfire/distributed/LocatorLauncher.java > 81d874d > > geode-core/src/main/java/com/gemstone/gemfire/distributed/SSLEnabledComponents.java > PRE-CREATION > > geode-core/src/main/java/com/gemstone/gemfire/distributed/ServerLauncher.java > cce482a > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/AbstractDistributionConfig.java > cc544f6 > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/DistributionConfig.java > 2ff6540 > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/DistributionConfigImpl.java > fd4743b > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/DistributionManager.java > 259f7c2 > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/InternalDistributedSystem.java > 49a4c97 > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/InternalLocator.java > 46d9610 > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/LonerDistributionManager.java > 7f8eed6 > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/RuntimeDistributionConfigImpl.java > c1384c3 > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/ServerLocation.java > 0e7430c > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/ServerLocator.java > f5b774f > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/StartupMessage.java > 82f5c88 > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/direct/DirectChannel.java > 9f18e62 > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/InternalDistributedMember.java > 2d8b8e1 > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/GMSMemberFactory.java > 4328bed > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/GMSUtil.java > d489431 > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/ServiceConfig.java > 6df0c7b > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/fd/GMSHealthMonitor.java > 9fdbb64 > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/locator/GMSLocator.java > 1065214 > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/membership/GMSJoinLeave.java > 58b794a > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/messenger/JGAddress.java > 1f97001 > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/messenger/JGroupsMessenger.java > a119bb5 > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/tcpserver/LocatorCancelException.java > PRE-CREATION > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/tcpserver/TcpClient.java > c770238 > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/tcpserver/TcpServer.java > ceb5af8 > geode-core/src/main/java/com/gemstone/gemfire/internal/AbstractConfig.java > 4d0c5a8 > geode-core/src/main/java/com/gemstone/gemfire/internal/AvailablePort.java > 7a8dccd > > geode-core/src/main/java/com/gemstone/gemfire/internal/DistributionLocator.java > 2549c7a > geode-core/src/main/java/com/gemstone/gemfire/internal/GemFireVersion.java > baf03b1 > geode-core/src/main/java/com/gemstone/gemfire/internal/MigrationClient.java > 2c81e90 > geode-core/src/main/java/com/gemstone/gemfire/internal/SocketCloser.java > 8318afa > geode-core/src/main/java/com/gemstone/gemfire/internal/SocketCreator.java > fded3c3 > geode-core/src/main/java/com/gemstone/gemfire/internal/SystemAdmin.java > e52950f > geode-core/src/main/java/com/gemstone/gemfire/internal/admin/SSLConfig.java > 36c553e > > geode-core/src/main/java/com/gemstone/gemfire/internal/admin/remote/DistributionLocatorId.java > 1459d89 > > geode-core/src/main/java/com/gemstone/gemfire/internal/admin/remote/FetchHostResponse.java > a9712be > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/CacheServerLauncher.java > 17875bf > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/ClusterConfigurationLoader.java > e362389 > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/GemFireCacheImpl.java > 76a7bad > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/partitioned/RedundancyLogger.java > f7e8621 > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/persistence/PersistentMemberPattern.java > a0e1f0a > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/tier/sockets/AcceptorImpl.java > 7422e16 > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/tier/sockets/CacheClientNotifier.java > 6fd4560 > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/tier/sockets/CacheClientUpdater.java > 26efd4d > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/tier/sockets/CacheServerStats.java > 658b35e > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/xmlcache/GatewayReceiverCreation.java > e76d39a > > geode-core/src/main/java/com/gemstone/gemfire/internal/i18n/LocalizedStrings.java > 443fe78 > > geode-core/src/main/java/com/gemstone/gemfire/internal/net/SSLConfigurationFactory.java > PRE-CREATION > > geode-core/src/main/java/com/gemstone/gemfire/internal/net/SSLEnabledComponent.java > PRE-CREATION > > geode-core/src/main/java/com/gemstone/gemfire/internal/net/SocketCloser.java > PRE-CREATION > > geode-core/src/main/java/com/gemstone/gemfire/internal/net/SocketCreator.java > PRE-CREATION > > geode-core/src/main/java/com/gemstone/gemfire/internal/net/SocketCreatorFactory.java > PRE-CREATION > > geode-core/src/main/java/com/gemstone/gemfire/internal/statistics/HostStatHelper.java > b52eaae > > geode-core/src/main/java/com/gemstone/gemfire/internal/statistics/HostStatSampler.java > d414d2c > > geode-core/src/main/java/com/gemstone/gemfire/internal/statistics/StatArchiveWriter.java > 027bde1 > geode-core/src/main/java/com/gemstone/gemfire/internal/tcp/Connection.java > a69f878 > > geode-core/src/main/java/com/gemstone/gemfire/internal/tcp/ConnectionTable.java > 3a5eb9f > geode-core/src/main/java/com/gemstone/gemfire/internal/tcp/TCPConduit.java > a38244c > > geode-core/src/main/java/com/gemstone/gemfire/management/GemFireProperties.java > 5f09d7d > > geode-core/src/main/java/com/gemstone/gemfire/management/internal/JettyHelper.java > 42dcd64 > > geode-core/src/main/java/com/gemstone/gemfire/management/internal/JmxManagerAdvisee.java > 3af34e1 > > geode-core/src/main/java/com/gemstone/gemfire/management/internal/JmxManagerLocatorRequest.java > 861f51d > > geode-core/src/main/java/com/gemstone/gemfire/management/internal/ManagementAgent.java > f3c771f > > geode-core/src/main/java/com/gemstone/gemfire/management/internal/RestAgent.java > e835bab > > geode-core/src/main/java/com/gemstone/gemfire/management/internal/beans/BeanUtilFuncs.java > b802648 > > geode-core/src/main/java/com/gemstone/gemfire/management/internal/beans/MemberMBeanBridge.java > 08260d3 > > geode-core/src/main/java/com/gemstone/gemfire/management/internal/cli/commands/LauncherLifecycleCommands.java > 7b04c06 > > geode-core/src/main/java/com/gemstone/gemfire/management/internal/cli/commands/ShellCommands.java > 09a25a6 > > geode-core/src/main/java/com/gemstone/gemfire/management/internal/cli/shell/JmxOperationInvoker.java > a67cae3 > > geode-core/src/main/java/com/gemstone/gemfire/memcached/GemFireMemcachedServer.java > ba0b479 > geode-core/src/main/java/org/apache/geode/redis/GeodeRedisServer.java > 6f6ae6d > > geode-core/src/test/java/com/gemstone/gemfire/cache/ConnectionPoolDUnitTest.java > fef2171 > > geode-core/src/test/java/com/gemstone/gemfire/cache/client/internal/AutoConnectionSourceImplJUnitTest.java > 42bd683 > > geode-core/src/test/java/com/gemstone/gemfire/cache/client/internal/CacheServerSSLConnectionDUnitTest.java > ee5cc62 > > geode-core/src/test/java/com/gemstone/gemfire/cache/client/internal/LocatorLoadBalancingDUnitTest.java > 1bc2322 > > geode-core/src/test/java/com/gemstone/gemfire/cache30/DistributedMulticastRegionDUnitTest.java > 3c05794 > > geode-core/src/test/java/com/gemstone/gemfire/distributed/AbstractLauncherIntegrationTestCase.java > cc13c48 > > geode-core/src/test/java/com/gemstone/gemfire/distributed/DistributedSystemDUnitTest.java > 1be3b9f > > geode-core/src/test/java/com/gemstone/gemfire/distributed/HostedLocatorsDUnitTest.java > 438c3f7 > > geode-core/src/test/java/com/gemstone/gemfire/distributed/LauncherMemberMXBeanIntegrationTest.java > 8c570ac > > geode-core/src/test/java/com/gemstone/gemfire/distributed/LocatorDUnitTest.java > 954846f > > geode-core/src/test/java/com/gemstone/gemfire/distributed/LocatorJUnitTest.java > af0c30d > > geode-core/src/test/java/com/gemstone/gemfire/distributed/LocatorLauncherLocalIntegrationTest.java > faf6d31 > > geode-core/src/test/java/com/gemstone/gemfire/distributed/LocatorLauncherRemoteIntegrationTest.java > 8e978e9 > > geode-core/src/test/java/com/gemstone/gemfire/distributed/ServerLauncherLocalIntegrationTest.java > 1baf48d > > geode-core/src/test/java/com/gemstone/gemfire/distributed/ServerLauncherRemoteIntegrationTest.java > 6d20673 > > geode-core/src/test/java/com/gemstone/gemfire/distributed/internal/DistributionConfigJUnitTest.java > 6d6f36d > > geode-core/src/test/java/com/gemstone/gemfire/distributed/internal/InternalDistributedSystemJUnitTest.java > a59262e > > geode-core/src/test/java/com/gemstone/gemfire/distributed/internal/StartupMessageDataJUnitTest.java > 6aa4248 > > geode-core/src/test/java/com/gemstone/gemfire/distributed/internal/membership/MembershipJUnitTest.java > 9c7180e > > geode-core/src/test/java/com/gemstone/gemfire/distributed/internal/membership/NetViewJUnitTest.java > fbc4189 > > geode-core/src/test/java/com/gemstone/gemfire/distributed/internal/membership/gms/fd/GMSHealthMonitorJUnitTest.java > d29553c > > geode-core/src/test/java/com/gemstone/gemfire/distributed/internal/membership/gms/locator/GMSLocatorRecoveryJUnitTest.java > fcf77a0 > > geode-core/src/test/java/com/gemstone/gemfire/distributed/internal/tcpserver/TcpServerBackwardCompatDUnitTest.java > 954aff4 > > geode-core/src/test/java/com/gemstone/gemfire/distributed/internal/tcpserver/TcpServerJUnitTest.java > 425b391 > > geode-core/src/test/java/com/gemstone/gemfire/internal/AbstractConfigJUnitTest.java > 400f872 > > geode-core/src/test/java/com/gemstone/gemfire/internal/JSSESocketJUnitTest.java > d0d906a > > geode-core/src/test/java/com/gemstone/gemfire/internal/SSLConfigJUnitTest.java > c8addf8 > > geode-core/src/test/java/com/gemstone/gemfire/internal/SocketCloserJUnitTest.java > cc71738 > > geode-core/src/test/java/com/gemstone/gemfire/internal/SocketCloserWithWaitJUnitTest.java > 0ba41ba > > geode-core/src/test/java/com/gemstone/gemfire/internal/net/JSSESocketJUnitTest.java > PRE-CREATION > > geode-core/src/test/java/com/gemstone/gemfire/internal/net/SSLConfigurationFactoryTest.java > PRE-CREATION > > geode-core/src/test/java/com/gemstone/gemfire/internal/net/SocketCloserJUnitTest.java > PRE-CREATION > > geode-core/src/test/java/com/gemstone/gemfire/internal/net/SocketCloserWithWaitJUnitTest.java > PRE-CREATION > > geode-core/src/test/java/com/gemstone/gemfire/internal/net/SocketCreatorFactoryJUnitTest.java > PRE-CREATION > > geode-core/src/test/java/com/gemstone/gemfire/internal/statistics/GemFireStatSamplerIntegrationTest.java > ff7c34d > > geode-core/src/test/java/com/gemstone/gemfire/internal/statistics/SimpleStatSamplerIntegrationTest.java > d86b968 > > geode-core/src/test/java/com/gemstone/gemfire/internal/tcp/ConnectionJUnitTest.java > 3a88707 > > geode-core/src/test/java/com/gemstone/gemfire/management/CacheManagementDUnitTest.java > 550a6cb > > geode-core/src/test/java/com/gemstone/gemfire/management/JMXMBeanDUnitTest.java > PRE-CREATION > > geode-core/src/test/java/com/gemstone/gemfire/management/UniversalMembershipListenerAdapterDUnitTest.java > 91a6188 > > geode-core/src/test/java/com/gemstone/gemfire/management/internal/JettyHelperJUnitTest.java > 1c66780 > > geode-core/src/test/java/com/gemstone/gemfire/management/internal/cli/commands/ConfigCommandsDUnitTest.java > 5720955 > > geode-core/src/test/java/com/gemstone/gemfire/management/internal/cli/commands/CreateAlterDestroyRegionCommandsDUnitTest.java > 21cdfbc > > geode-core/src/test/java/com/gemstone/gemfire/management/internal/cli/commands/DeployCommandsDUnitTest.java > 6f70e57 > > geode-core/src/test/java/com/gemstone/gemfire/management/internal/cli/commands/DiskStoreCommandsDUnitTest.java > 32cea6a > > geode-core/src/test/java/com/gemstone/gemfire/management/internal/cli/commands/HTTPServiceSSLSupportJUnitTest.java > 7a96adf > > geode-core/src/test/java/com/gemstone/gemfire/management/internal/cli/commands/IndexCommandsDUnitTest.java > 2691e09 > > geode-core/src/test/java/com/gemstone/gemfire/management/internal/cli/commands/QueueCommandsDUnitTest.java > 86c61b6 > > geode-core/src/test/java/com/gemstone/gemfire/management/internal/cli/commands/SharedConfigurationCommandsDUnitTest.java > 87a7ab9 > > geode-core/src/test/java/com/gemstone/gemfire/management/internal/configuration/SharedConfigurationDUnitTest.java > 3c17f9f > > geode-core/src/test/java/com/gemstone/gemfire/security/generator/SSLCredentialGenerator.java > c591732 > geode-core/src/test/java/com/gemstone/gemfire/test/dunit/NetworkUtils.java > dec882e > > geode-core/src/test/java/com/gemstone/gemfire/test/dunit/internal/JUnit4DistributedTestCase.java > 686779d > geode-core/src/test/java/org/apache/geode/redis/RedisDistDUnitTest.java > eb87797 > > geode-core/src/test/resources/com/gemstone/gemfire/codeAnalysis/excludedClasses.txt > ddacf99 > > geode-core/src/test/resources/com/gemstone/gemfire/codeAnalysis/sanctionedSerializables.txt > 03288c2 > > geode-core/src/test/resources/com/gemstone/gemfire/internal/net/multiKey.jks > PRE-CREATION > > geode-core/src/test/resources/com/gemstone/gemfire/internal/net/multiKeyTrust.jks > PRE-CREATION > > geode-pulse/src/test/java/com/vmware/gemfire/tools/pulse/testbed/driver/PulseUITest.java > 24ba815 > > geode-pulse/src/test/java/com/vmware/gemfire/tools/pulse/tests/PulseAbstractTest.java > eeaa069 > > geode-wan/src/main/java/com/gemstone/gemfire/cache/client/internal/locator/wan/LocatorDiscovery.java > 0fd206e > > geode-wan/src/main/java/com/gemstone/gemfire/cache/client/internal/locator/wan/LocatorMembershipListenerImpl.java > 5f4943d > > geode-wan/src/main/java/com/gemstone/gemfire/internal/cache/wan/AbstractRemoteGatewaySender.java > d45d5ef > > geode-wan/src/main/java/com/gemstone/gemfire/internal/cache/wan/GatewayReceiverImpl.java > 9fd73e6 > > geode-wan/src/test/java/com/gemstone/gemfire/internal/cache/wan/WANTestBase.java > fd2c0b5 > > geode-wan/src/test/java/com/gemstone/gemfire/management/internal/configuration/ClusterConfigurationDUnitTest.java > bdd5d71 > > geode-web-api/src/main/java/com/gemstone/gemfire/rest/internal/web/swagger/config/RestApiPathProvider.java > b35dce9 > gradle/java.gradle 36d88dc > > Diff: https://reviews.apache.org/r/51227/diff/ > > > Testing > ------- > > precheckin > regression testing to follow > > > Thanks, > > Udo Kohlmeyer > >