Uploaded image for project: 'Chemistry'
  1. Chemistry
  2. CMIS-809

cmislib applyACL method has multiple issues

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Open
    • Major
    • Resolution: Unresolved
    • cmislib 0.5.1
    • None
    • python-cmislib
    • None

    Description

      The testApplyACL() unit test is failing against Apache Chemistry 0.11.0. In addition, certain use cases are not handled properly, such as adding two different permissions for the same principal. Currently only the last one in the ACL is applied.

      This was originally implemented by accepting an ACL object that represented the ACL the caller wanted on the document, then doing a PUT against the object's ACL.

      On further review of the spec, the applyACL method should be refactored to accept an add list and a remove list. As part of this, the case where duplicate principals are added to the same list needs to be incorporated.

      The unit test could also be beefed up.

      Attachments

        Activity

          People

            Unassigned Unassigned
            jpotts Jeffrey Taylor Potts
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated: