Qpid
  1. Qpid
  2. QPID-4678

[Java Broker] add ACL to restrict new support for broker-level configuration through the management interface

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 0.21
    • Fix Version/s: 0.22
    • Component/s: Java Broker
    • Labels:
      None

      Description

      Recent changes have introduced functionality to configure the broker more extensively through the REST interface. E.g configuring the broker attributes for things like the ACL file, group file, ssl settings, and also adding/removing/editing things like virtualhosts, ports ,authentication providers, etc.

      ACL options needs to be added to ensure this functionality can be locked down.

        Activity

        Hide
        Alex Rudyy added a comment -

        Robbie,
        Please review changes in commit http://svn.apache.org/r1462551 .

        Show
        Alex Rudyy added a comment - Robbie, Please review changes in commit http://svn.apache.org/r1462551 .
        Hide
        Robbie Gemmell added a comment -

        Changes look good to me.

        Will request for 0.22, but leave JIRA in RtR to indicate its not done (given the blocker priority).

        Show
        Robbie Gemmell added a comment - Changes look good to me. Will request for 0.22, but leave JIRA in RtR to indicate its not done (given the blocker priority).
        Hide
        Justin Ross added a comment -

        Reviewed by Robbie. Approved for 0.22.

        Show
        Justin Ross added a comment - Reviewed by Robbie. Approved for 0.22.
        Hide
        Robbie Gemmell added a comment -

        r1462551 was merged in http://svn.apache.org/r1463170

        Show
        Robbie Gemmell added a comment - r1462551 was merged in http://svn.apache.org/r1463170
        Hide
        Alex Rudyy added a comment -

        Reopening the JIRA as the ACL checks are performed twice

        Show
        Alex Rudyy added a comment - Reopening the JIRA as the ACL checks are performed twice
        Hide
        Alex Rudyy added a comment -

        Robbie,
        Could you please review the changes in revision http://svn.apache.org/r1463934 ?

        Show
        Alex Rudyy added a comment - Robbie, Could you please review the changes in revision http://svn.apache.org/r1463934 ?
        Hide
        Robbie Gemmell added a comment -

        Looks good to me. We should ask to include this in 0.22.

        Show
        Robbie Gemmell added a comment - Looks good to me. We should ask to include this in 0.22.
        Hide
        Justin Ross added a comment -

        Reviewed by Robbie. Approved for 0.22.

        Show
        Justin Ross added a comment - Reviewed by Robbie. Approved for 0.22.
        Hide
        Alex Rudyy added a comment -

        The changes in revision r1463934 were applied to 0.22 branch in revision http://svn.apache.org/r1464494

        Show
        Alex Rudyy added a comment - The changes in revision r1463934 were applied to 0.22 branch in revision http://svn.apache.org/r1464494

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development