Uploaded image for project: 'Grouper'
  1. Grouper
  2. GRP-4655

Viewer group for provisioner checks for view privilege in the group, not membership

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • 4.1.1
    • 2.6.14
    • provisioning
    • None

    Description

      Is this change correct? You should need membership in the ACL group, not a view privilege, right?

      commit 95bac8e577cf8edb211d7f55dec5acf96f557874
      Date: Tue Aug 9 22:56:19 2022 -0700
       
      Show provisioning button for viewers as well
       
      — a/grouper/src/grouper/edu/internet2/middleware/grouper/app/provisioning/GrouperProvisioningService.java
      +++ b/grouper/src/grouper/edu/internet2/middleware/grouper/app/provisioning/GrouperProvisioningService.java
       
      @@ -1624,7 +1625,7 @@ public class GrouperProvisioningService {
                 }
               }
               
      - return group.hasMember(subject);
      + return group.hasPrivilege(subject, AccessPrivilege.VIEW.getName());
             }
      

      Attachments

        Activity

          People

            chris.hyzer@at.internet2.edu Chris Hyzer (upenn.edu)
            chad.redman.3@at.internet2.edu Chad Redman
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: