Qpid
  1. Qpid
  2. QPID-3099

Java qpid-client ignores hostnames containing underscores

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.5, 0.6, 0.8
    • Fix Version/s: 0.10
    • Component/s: Java Client
    • Labels:
      None

      Description

      When defining a connection URL listing a broker that has an underscore the hostname results in the broker host being evaluated to "localhost" rather than the intended host.

      e.g. amqp://guest:guest@clientid/test?brokerlist='tcp://under_score:5672'
      amqp://guest:guest@clientid/test?brokerlist='tcp://under_score'

        Activity

        Hide
        Robbie Gemmell added a comment -

        An underscore is not a legal hostname character, so im not sure this is a bug exactly? Perhaps logging the presence of an illegal hostname would be helpful though.

        Show
        Robbie Gemmell added a comment - An underscore is not a legal hostname character, so im not sure this is a bug exactly? Perhaps logging the presence of an illegal hostname would be helpful though.
        Hide
        Rajith Attapattu added a comment -

        I treated this as a bug based on a customer request.
        My first impression was that an underscore was a bit odd in a host name, but it seems they have named a host by that name.
        I would leave this fix as it is and leave the correct naming of hosts to the end users.

        Show
        Rajith Attapattu added a comment - I treated this as a bug based on a customer request. My first impression was that an underscore was a bit odd in a host name, but it seems they have named a host by that name. I would leave this fix as it is and leave the correct naming of hosts to the end users.
        Hide
        Rajith Attapattu added a comment -

        Fixed along with a test case.

        Show
        Rajith Attapattu added a comment - Fixed along with a test case.

          People

          • Assignee:
            Rajith Attapattu
            Reporter:
            Rajith Attapattu
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development