Self-registration property ExpireAfterDays used to set enabled date not disabled

XMLWordPrintable

    • Type: Bug
    • Resolution: Fixed
    • Priority: Minor
    • 4.14.0, 5.11.0
    • Affects Version/s: 2.2.0
    • Component/s: API
    • None

      externalSubjects.autoaddGroupExpireAfterDays = 7
      externalSubjects.autoaddGroups = test:autoProvisionGroup
      externalSubjects.registerRequiresInvite = false

       

      edu.internet2.middleware.grouper.externalSubjects.ExternalSubject.assignGroups(String groups, String actions, boolean isInsert, final int expireAfterDays)

      ...
                Group group = GroupFinder.findByName(grouperSession, groupName, true);
                group.addMember(subject, false);
                if (expireAfterDays > 0) {
                  Membership membership = group.getImmediateMembership(Group.getDefaultList(), subject, true, true);
              **  membership.setEnabledTime(new Timestamp(System.currentTimeMillis() + ((long)expireAfterDays*24*60*60*1000)));
                  membership.update();
                }
      ...
      

      This probably needs to be membership.setDisabledTime, right? When auto provisioning, memberships are set to start 7 days in the future, which doesn't make sense.

            Assignee:
            Shilen Patel
            Reporter:
            Chad Redman
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: