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: Closed
    • 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

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        1160d 16m 1 Martin Ritchie 25/May/10 15:02
        Resolved Resolved Closed Closed
        1723d 5h 4m 1 Rob Godfrey 11/Feb/15 20:06
        Rob Godfrey made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Robbie Gemmell made changes -
        Component/s Starter [ 12312518 ]
        Robbie Gemmell made changes -
        Labels starter
        Martin Ritchie made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 0.5 [ 12313597 ]
        Resolution Fixed [ 1 ]
        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.
        Martin Ritchie made changes -
        Fix Version/s M5 [ 12313597 ]
        Marnie McCormack made changes -
        Priority Minor [ 4 ] Major [ 3 ]
        Assignee Martin Ritchie [ ritchiem ]
        Fix Version/s M5 [ 12313597 ]
        Marnie McCormack made changes -
        Fix Version/s M4 [ 12313279 ]
        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
        Martin Ritchie made changes -
        Assignee Martin Ritchie [ ritchiem ]
        Marnie McCormack made changes -
        Component/s Java Broker [ 12311388 ]
        Marnie McCormack made changes -
        Component/s Java Broker [ 12311388 ]
        Component/s Starter [ 12312518 ]
        Aidan Skinner made changes -
        Fix Version/s M4 [ 12313279 ]
        Fix Version/s M3 [ 12312117 ]
        Gavin made changes -
        Workflow QPid [ 12437934 ] QPid Workflow [ 12439381 ]
        Gavin made changes -
        Workflow jira [ 12400202 ] QPid [ 12437934 ]
        Aidan Skinner made changes -
        Assignee Martin Ritchie [ ritchiem ]
        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
        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
        Martin Ritchie made changes -
        Field Original Value New Value
        Fix Version/s M3 [ 12312117 ]
        Priority Major [ 3 ] Minor [ 4 ]
        Martin Ritchie created issue -

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development