Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Azure AD Patch request issues #75

Closed
plamenGo opened this issue Jun 26, 2020 · 3 comments
Closed

Azure AD Patch request issues #75

plamenGo opened this issue Jun 26, 2020 · 3 comments

Comments

@plamenGo
Copy link

Hey, Azure posts the following when trying to PATCH a user:

{
"schemas":["urn:ietf:params:scim:api:messages:2.0:PatchOp"],
"Operations":[{
        "op":"Add","path":"urn:ietf:params:scim:schemas:extension:enterprise:2.0:User:manager",
        "value":"45619541-95de-4d5e-9872-571b5d2c5577"}]
}

We are unable to parse and the get following error from go-scim:

{
    "schemas": [
        "urn:ietf:params:scim:api:messages:2.0:Error"
    ],
    "status": 400,
    "scimType": "invalidPath",
    "detail": "invalidPath: error compiling path"
}

I think the issue might be that manager is a complex attribute under the enterprise schema, but is sent over with a simple ID.

Do you know if this is a bug in go-scim of if we should format the PATCH request differently? Can you provide an example of what this request should look like so it works in go-scim.

Best Regards!
Plamen

@plamenGo
Copy link
Author

Same thing happens with the employee number field.

{"schemas":["urn:ietf:params:scim:api:messages:2.0:PatchOp"],"Operations":[{"op":"Add","path":"urn:ietf:params:scim:schemas:extension:enterprise:2.0:User:employeeNumber","value":"6546579"}]}

This is not a complex field, so this leads me to believe that we're just not processing the path correctly

@plamenGo
Copy link
Author

After digging some more it seems that the path compiler does not like the colons (:) in the path name -- it's not clear to me what I should do about it though. I suspect the (.) may trip it up as well. Any advice would be welcome!

@imulab
Copy link
Owner

imulab commented Jun 27, 2020

@plamenGo looks like a very legit request, will take a look at this the day after.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants