Qpid
  1. Qpid
  2. QPID-1612

Split Java broker configuration into a number of files to allow validation.

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: M4
    • Fix Version/s: JIRA Cleanup
    • Component/s: Java Broker
    • Labels:
      None

      Description

      As a first step towards improving the Java broker configuration as per http://cwiki.apache.org/confluence/display/qpid/Java+Broker+Configuration+Design

      Utilise the ConfigurationFactory code provided by Commons Configuration to allow the splitting out of the virtualhost configuration and so allowing the validation of the other sections. This would mean our initial config.xml file would be come something like this:

      <?xml version="1.0" encoding="ISO-8859-1" ?>
      <configuration>
      <xml fileName="$

      {QPID_HOME}/etc/environment.xml" validating="false"/>
      <-xml fileName="${QPID_HOME}

      /etc/user-config.xml" validating="false" optional="true"/>
      <xml fileName="$

      {QPID_HOME}/etc/main-config.xml" validating="false"/>
      <additional>
      <xml fileName="${QPID_HOME}

      /etc/virtualhosts.xml" validating="false" at="virtualhosts"/>
      </additional>

      </configuration>

        Activity

        Hide
        Robbie Gemmell added a comment -

        Review-OK'ing issue as part of JIRA cleanup. Issue could be resolved, but may not be: see QPID-3469 for further details.

        Show
        Robbie Gemmell added a comment - Review-OK'ing issue as part of JIRA cleanup. Issue could be resolved, but may not be: see QPID-3469 for further details.
        Hide
        Martin Ritchie added a comment -

        We should also make the <name>...</name> of the default Principal Database clearer. Currently the text passwordfile appears in several locations and will confuse users. Suggest we change it to 'default-principal-database' then it should be clearer that the entry in the <jmx> section is referencing the <PrincipalDatabase> definition.

        Show
        Martin Ritchie added a comment - We should also make the <name>...</name> of the default Principal Database clearer. Currently the text passwordfile appears in several locations and will confuse users. Suggest we change it to 'default-principal-database' then it should be clearer that the entry in the <jmx> section is referencing the <PrincipalDatabase> definition.

          People

          • Assignee:
            Unassigned
            Reporter:
            Martin Ritchie
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development