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

lite add member web service should accept subjectIdentifier

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • 2.6.17
    • None
    • None
    • None

    Description

      Liam Hoekenga
      3 days ago
      We tried a payload like this…
      {
      'WsRestAddMemberLiteRequest':

      { 'subjectIdentifier': 'bjensen', 'groupName': 'app:MicrosquishAzul:service:ref:enrolled' }

      }

      Liam Hoekenga
      3 days ago
      and the error log shows
      2022-09-27T17:28:00,648: [ajp-nio-0.0.0.0-8009-exec-4] ERROR GrouperRestServlet.service(226) - [] - Problem with request: uri: /grouper-ws/servicesRest/v2_6_000/groups/app:MicrosquishAzul:service:ref:enrolled/members/, method: POST, decoded url strings: 0: 'v2_6_000', 1: 'groups', 2: 'groupName': 'app:MicrosquishAzul:service:ref:enrolled', 3: 'members'
      edu.internet2.middleware.grouper.ws.exceptions.WsInvalidQueryException: The field 'subjectId' is required
      (edited)

      Liam Hoekenga
      3 days ago
      if we’re passing subjectIdentifier, do we need to send it a null subjectId ?

      Liam Hoekenga
      3 days ago
      The soap request from the presention suggests that groupName and subjectIdentifier should be enough?

      Liam Hoekenga
      2 days ago
      @mchyzer
      is this a bug?

      Chris Hyzer
      14 minutes ago
      this works:
      https://grouperws.server.edu/grouperWs/servicesRest/2.6.13/groups
      {
      "WsRestAddMemberRequest":{
      "wsGroupLookup":

      { "groupName":"test:testGroup" }

      ,
      "subjectLookups":[

      { "subjectIdentifier":"mchyzer" }

      ]
      }
      }
      :ty-thank-you:
      1

      New

      Liam Hoekenga
      13 minutes ago
      Did I misread the SOAP example?

      Chris Hyzer
      11 minutes ago
      in the "lite" service, for rest, it is requiring subjectId. this is the non-lite, but thats ok right? :slightly_smiling_face:

      Liam Hoekenga
      11 minutes ago
      that’s fine.

      Chris Hyzer
      11 minutes ago
      ok cool

      Liam Hoekenga
      10 minutes ago
      so the lite version for SOAP accepts subjectIdentifier but the REST version doesn’t?

      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: