James Server
  1. James Server
  2. JAMES-932

James 'initialize' reports deprecated config <usernames> <servernames>

    Details

      Description

      During start log messages such as

      <usernames> parameter in James block is deprecated. Please configure this data in UsersRepository block: configuration injected for backward compatibility

      and

      <servernames> parameter in James block is deprecated. Please configure this data in domainlist block: configuration injected for backward compatibility

      The code in org.apache.james.James.initialize()
      uses the following test conditions

      Configuration userNamesConf = conf.getChild("usernames");
      if (userNamesConf != null)

      { ..}

      Configuration serverConf = conf.getChild("servernames");
      if (serverConf != null)

      { .. }

      The javadocs for org.apache.avalon.framework.configuration.Configuration state
      a test for null should use getChild(String, boolean)

      The current james-config.xml has these listed as deprecated and remd out.
      For this test condition, the method should return null.

      I believe the following would be correct for this situation.

      userNamesConf = conf.getChild("usernames",false);

      and

      Configuration serverConf = conf.getChild("servernames", false);

      The following could be an issue, but they are still valid in james-config.xml.

      virtualHostingConfig = conf.getChild("enableVirtualHosting");
      defaultDomainConfig = conf.getChild("defaultDomain");
      helloNameConfig = conf.getChild("helloName");

        Activity

        Hide
        Norman Maurer added a comment -

        I think thats not a problem anymore because we use commons-configuration now

        Show
        Norman Maurer added a comment - I think thats not a problem anymore because we use commons-configuration now

          People

          • Assignee:
            Norman Maurer
            Reporter:
            Mark DeBusschere
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 10m
              10m
              Remaining:
              Remaining Estimate - 10m
              10m
              Logged:
              Time Spent - Not Specified
              Not Specified

                Development