• Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • 4.1.1, 4.2.0
    • 4.0.6, 4.1.0
    • None
    • None
    • 4.2 M1

      Some AD group can contain #, we should support it

          [RUNTIME-2772] A group can not contains # in it id

          To be modified :

          • org.ametys.core.group.GroupIdentity#stringToGroupIdentity
          • org.ametys.core.user.UserIdentity#stringToUserIdentity
          • org.ametys.plugins.core.group.GroupSearchAction#_searchGroupsByContext
          • org.ametys.plugins.core.user.UserSearchAction#_searchUsersByContext
          • widget edition.group

          In all cases, we user String#split and we could change this by StringUtils.substringBeforeLast("#") StringUtils.substringAfterLast("#") to allow the '#' character in login/groupId (as we know that '#' is forbiddent in user and group populations)

          Laurence Aumeunier added a comment - To be modified : org.ametys.core.group.GroupIdentity#stringToGroupIdentity org.ametys.core.user.UserIdentity#stringToUserIdentity org.ametys.plugins.core.group.GroupSearchAction#_searchGroupsByContext org.ametys.plugins.core.user.UserSearchAction#_searchUsersByContext widget edition.group In all cases, we user String#split and we could change this by StringUtils.substringBeforeLast("#") StringUtils.substringAfterLast("#") to allow the '#' character in login/groupId (as we know that '#' is forbiddent in user and group populations)

            laurence Laurence Aumeunier
            laurence Laurence Aumeunier
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: