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

group name hooks not registered in GSH, but are registered in WS and UI

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • None
    • 2.3.1, 2.4.0, 2.3.0.patch
    • API
    • None

    Description

      ----Original Message----
      From: grouper-users-request@internet2.edu grouper-users-request@internet2.edu On Behalf Of Kumi Hagimoto
      Sent: Thursday, May 18, 2017 4:27 PM
      To: grouper-users@internet2.edu
      Subject: [grouper-users] Regex Validator ignored when testing via GSH

      Hello,

      I have the following regex validator in grouper.properties:

      group.attribute.validator.attributeName.0=extension
      group.attribute.validator.regex.0=^[a-z0-9\.-]+$
      group.attribute.validator.vetoMessage.0=Group ID '$attributeValue$' is invalid
      since it must contain only lowercase letters, numbers, hyphens, and periods

      It works in UI and WS, but not when I test it via GSH. I do have an extra
      group hook defined, but that's true for UI/WS as well. (And GSH does honor
      that extra hook..)

      If I try to add a group with extension "TOG1" via UI, I get an error message:
      "Group ID 'TOG1' is invalid since it must contain only lowercase letters,
      numbers, hyphens, and periods" which is expected. But I can do
      addGroup("osu:org:sg:to","TOG1","Test Org Group 1"); via GSH and it will add
      the group - no error.

      I'm not sure what I'm doing wrong... any ideas? I use GSH to test thing
      sometimes, so it would be nice if I could get the same results.

      Thanks,
      Kumi

      Attachments

        Activity

          People

            chris.hyzer@at.internet2.edu Chris Hyzer (upenn.edu)
            chris.hyzer@at.internet2.edu Chris Hyzer (upenn.edu)
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Smart Checklist