Qpid
  1. Qpid
  2. QPID-4747

[Java Broker] make specifying the authentication provider mandatory on ports that use them and remove the defaultAuthenticationProvider attribute on the broker

    Details

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

      Description

      Make specifying the authentication provider mandatory on each port type that requires it and remove the defaultAuthenticationProvider attribute on the broker.

      The AMQP, JMX, HTTP ports are authenticated but the RMI registry port used to advertise the JMX connector isn't and cant make use of an authentication provider. The defaultAuthenticationProvider attribute makes this unclear, and uses some fairly ugly wiring up code at broker startup to make work, it should be removed and the currently optional per-port authenticationProvider attribute made mandatory on the port types that require it.

        Issue Links

          Activity

          Hide
          Robbie Gemmell added a comment -

          Tweak made via http://svn.apache.org/r1479839 and merged to the 0.22 release branch via http://svn.apache.org/r1479843

          Show
          Robbie Gemmell added a comment - Tweak made via http://svn.apache.org/r1479839 and merged to the 0.22 release branch via http://svn.apache.org/r1479843
          Hide
          Robbie Gemmell added a comment -

          r1468830 merged to 0.22 branch via: http://svn.apache.org/r1469915
          r1468875 merged to 0.22 branch via: http://svn.apache.org/r1469918
          r1468953 merged to 0.22 branch via: http://svn.apache.org/r1469920
          r1469256 merged to 0.22 branch via: http://svn.apache.org/r1469922

          Show
          Robbie Gemmell added a comment - r1468830 merged to 0.22 branch via: http://svn.apache.org/r1469915 r1468875 merged to 0.22 branch via: http://svn.apache.org/r1469918 r1468953 merged to 0.22 branch via: http://svn.apache.org/r1469920 r1469256 merged to 0.22 branch via: http://svn.apache.org/r1469922
          Hide
          Robbie Gemmell added a comment -

          Change committed by Alex, worked on by both of us, reviewed by both of us:
          http://svn.apache.org/r1468830

          Leaving JIRA open until merged to 0.22 branch

          Show
          Robbie Gemmell added a comment - Change committed by Alex, worked on by both of us, reviewed by both of us: http://svn.apache.org/r1468830 Leaving JIRA open until merged to 0.22 branch

            People

            • Assignee:
              Robbie Gemmell
              Reporter:
              Robbie Gemmell
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development