Qpid
  1. Qpid
  2. QPID-3249

Remove incomplete support for authentication at virtualhost level

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.10
    • Fix Version/s: 0.11
    • Component/s: Java Broker
    • Labels:
      None

      Description

      In addition to the principal db specified at the broker level, VirtualHostImpl tries to support authentication at virtualhost level. This concept is flawed as in Qpid, username and password must be considered, before virtual host.

      i.e.

      <virtualhost>
       <name>localhost</name>
       <localhost>
      ....
         <security>
           <!-- Would need protocol changes to allow this-->
           <authentication>
             <name>passwordfile</name>
             <!-- Currently this can't be used as Vhost isn't specified at connection start only connection open -->
             <mechanism>PLAIN</mechanism>
           </authentication>
      .....
      
      

      The redundant implementation in VirtualHostImpl should be removed.

        Activity

        Hide
        Keith Wall added a comment -

        Attached patch. This patch refactors VirtualHostImpl to remove its dependency on PrincipalAuthenticationManager. Added unit tests to PrincipalAuthenticationManager to support the change.

        Show
        Keith Wall added a comment - Attached patch. This patch refactors VirtualHostImpl to remove its dependency on PrincipalAuthenticationManager. Added unit tests to PrincipalAuthenticationManager to support the change.
        Hide
        Keith Wall added a comment -

        Hi Robbie - would you mind reviewing please?

        Show
        Keith Wall added a comment - Hi Robbie - would you mind reviewing please?
        Hide
        Robbie Gemmell added a comment -

        Patch applied.

        Show
        Robbie Gemmell added a comment - Patch applied.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development