Uploaded image for project: 'Sling'
  1. Sling
  2. SLING-9971

AclManagerTest/RepPolicyEntryHandlerTest : no tests for 'deny' entries

    XMLWordPrintableJSON

Details

    Description

      from what i can see there exists not a single test case for 'deny' access control entries. while i agree that creating deny-entries for system users should be considered bad practice, it's it possible with resource-based access control setup (note though that principal-based access control setup only allows for 'allow' entries, see http://jackrabbit.apache.org/api/2.18/org/apache/jackrabbit/api/security/authorization/PrincipalAccessControlList.html#addEntry-java.lang.String-javax.jcr.security.Privilege:A- and http://jackrabbit.apache.org/oak/docs/security/authorization/principalbased.html#Implementation_Details).

      unless the converter intended to prevent 'deny' entries from being used (currently not the case), i think there should be at least 1 test that verifies that deny entries will be properly converted.

      Attachments

        Issue Links

          Activity

            People

              karlpauls Karl Pauls
              angela Angela Schreiber
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: