Qpid
  1. Qpid
  2. QPID-431

Main startup of virtualhosts doesn't correctly resolve any config file variables such as ${conf}

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.5
    • Component/s: Java Broker
    • Labels:

      Description

      Main.setupVirtualHosts/startup

      Doesn't correctly resolve the path from the virtualhosts section of the XML. Currently it resolved $

      {conf}

      manually to be the location of the config.xml file and not the value specified in the xml file.

      This should be resolved by the Configuration classes.

        Activity

        Hide
        Aidan Skinner added a comment -

        This manifests itself with qpid -c config.xml failing with 'could not open null/virtualhosts.xml' but qpid -c ./config.xml working

        Show
        Aidan Skinner added a comment - This manifests itself with qpid -c config.xml failing with 'could not open null/virtualhosts.xml' but qpid -c ./config.xml working
        Hide
        Martin Ritchie added a comment -

        To be accurate the $

        {conf}

        value used in the <virtualhost> section of the xml resolves to be the basename of config.xml. So if there is no basename the value java returns is null hence null/virtualhosts.xml.

        It is just poor the ${conf] value should be looked up in the Configuration class as is done for the rest of the config.xml file.

        Show
        Martin Ritchie added a comment - To be accurate the $ {conf} value used in the <virtualhost> section of the xml resolves to be the basename of config.xml. So if there is no basename the value java returns is null hence null/virtualhosts.xml. It is just poor the ${conf] value should be looked up in the Configuration class as is done for the rest of the config.xml file.
        Hide
        Marnie McCormack added a comment -

        Moving items not being worked on afaik out of M4 Fix Version

        Show
        Marnie McCormack added a comment - Moving items not being worked on afaik out of M4 Fix Version
        Hide
        Martin Ritchie added a comment -

        This was adddressed when QPID-1612 was done.

        Show
        Martin Ritchie added a comment - This was adddressed when QPID-1612 was done.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development