mirror of https://github.com/OpenIdentityPlatform/OpenDJ.git

jvergara
29.20.2008 41898f409cc68b72a53a1842c914c368d906ac3c
Fix for issue 3537 (QuickSetup proposed SSL port is random...)

Use the LDAP port instead of the admin connector port to calculate the default SSL port.
2 files modified
32 ■■■■ changed files
opendj-sdk/opends/src/quicksetup/org/opends/quicksetup/UserData.java 14 ●●●● patch | view | raw | blame | history
opendj-sdk/opends/src/server/org/opends/server/tools/InstallDSArgumentParser.java 18 ●●●● patch | view | raw | blame | history
opendj-sdk/opends/src/quicksetup/org/opends/quicksetup/UserData.java
@@ -113,17 +113,17 @@
    setNewSuffixOptions(defaultNewSuffixOptions);
    // See what we can propose as port
    int defaultPort = getDefaultPort();
    if (defaultPort != -1)
    int defaultLdapPort = getDefaultPort();
    if (defaultLdapPort != -1)
    {
      setServerPort(defaultPort);
      setServerPort(defaultLdapPort);
    }
//  See what we can propose as port
    defaultPort = getDefaultAdminConnectorPort();
    if (defaultPort != -1)
    int defaultAdminPort = getDefaultAdminConnectorPort();
    if (defaultAdminPort != -1)
    {
      setAdminConnectorPort(defaultPort);
      setAdminConnectorPort(defaultAdminPort);
    }
    setHostName(getDefaultHostName());
@@ -142,7 +142,7 @@
          new HashSet<SuffixDescriptor>());
    setSuffixesToReplicateOptions(suffixes);
    SecurityOptions sec = SecurityOptions.createNoCertificateOptions();
    sec.setSslPort(getDefaultSslPort(defaultPort));
    sec.setSslPort(getDefaultSslPort(defaultLdapPort));
    setSecurityOptions(sec);
    remoteWithNoReplicationPort =
opendj-sdk/opends/src/server/org/opends/server/tools/InstallDSArgumentParser.java
@@ -240,28 +240,28 @@
        INFO_INSTALLDS_DESCRIPTION_SAMPLE_DATA.get());
    addArgument(sampleDataArg);
    int defaultPort = UserData.getDefaultPort();
    if (defaultPort == -1)
    int defaultLdapPort = UserData.getDefaultPort();
    if (defaultLdapPort == -1)
    {
      defaultPort = 389;
      defaultLdapPort = 389;
    }
    ldapPortArg = new IntegerArgument(
        "ldapPort".toLowerCase(), OPTION_SHORT_PORT,
        "ldapPort", false, false,
        true, INFO_PORT_PLACEHOLDER.get(), defaultPort,
        true, INFO_PORT_PLACEHOLDER.get(), defaultLdapPort,
        "ldapPort", true, 1, true, 65535,
        INFO_INSTALLDS_DESCRIPTION_LDAPPORT.get());
    addArgument(ldapPortArg);
    defaultPort = UserData.getDefaultAdminConnectorPort();
    if (defaultPort == -1)
    int defaultAdminPort = UserData.getDefaultAdminConnectorPort();
    if (defaultAdminPort == -1)
    {
      defaultPort = 4444;
      defaultAdminPort = 4444;
    }
    adminConnectorPortArg = new IntegerArgument(
        "adminConnectorPort".toLowerCase(), null,
        "adminConnectorPort", false, false,
        true, INFO_PORT_PLACEHOLDER.get(), defaultPort,
        true, INFO_PORT_PLACEHOLDER.get(), defaultAdminPort,
        "adminConnectorPort", true, 1, true, 65535,
        INFO_INSTALLDS_DESCRIPTION_ADMINCONNECTORPORT.get());
    addArgument(adminConnectorPortArg);
@@ -329,7 +329,7 @@
    enableStartTLSArg.setPropertyName("enableStartTLS");
    addArgument(enableStartTLSArg);
    int defaultSecurePort = UserData.getDefaultSslPort(defaultPort);
    int defaultSecurePort = UserData.getDefaultSslPort(defaultLdapPort);
    if (defaultSecurePort == -1)
    {
      defaultSecurePort = 636;