Uploaded image for project: 'Chukwa'
  1. Chukwa
  2. CHUKWA-507

JMSAdaptor url validation too strict

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Data Collection
    • Labels:
      None

      Description

      the JMSAdaptor requires that all JMS broker urls start with 'tcp://' or 'vm://', which is too strict. ActiveMQ has another transport that starts with 'failover://' for example.

        Activity

        Hide
        billgraham Bill Graham added a comment -

        Attaching patch that just requires '://' to be present.

        Show
        billgraham Bill Graham added a comment - Attaching patch that just requires '://' to be present.
        Hide
        eyang Eric Yang added a comment -

        +1 looks good.

        Show
        eyang Eric Yang added a comment - +1 looks good.
        Hide
        eyang Eric Yang added a comment -

        I just committed this, thanks Bill.

        Show
        eyang Eric Yang added a comment - I just committed this, thanks Bill.

          People

          • Assignee:
            billgraham Bill Graham
            Reporter:
            billgraham Bill Graham
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development