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

Add support for PATCH verb to service #29

Closed
jgomer2001 opened this Issue Sep 12, 2017 · 2 comments

Comments

Projects
None yet
1 participant
@jgomer2001
Collaborator

jgomer2001 commented Sep 12, 2017

Currently SCIM service does not officially support PATCH operation. According to spec, patching is one of the mechanisms that allows a SCIM client to remove (nullify) an already non-empty attribute.

PUT operation MAY cope for attribute removal (see RFC 7644 section 3.5.1). In current implementation an unassigned/nulled attribute passed to the service simply leaves the stored attribute value intact. IMO this is right: clearing attributes not being passed can lead to unintentional info erasure, and thus, complaints.

Current scim server code in oxTrust for patching is not complete yet and deviates from spec. Needs work.

@jgomer2001

This comment has been minimized.

Collaborator

jgomer2001 commented Nov 12, 2017

A new partial implementation for PATCH has been developed.

This impl supports add/replace/remove operations and all behaviors as per spec, excluding support for value selection ("valuePath") filters.

This implies that there is no way to delete or change a specific item from a multivalued attribute. So all subattributes within multi-valued attributes are being patched (be it addition, replacement, or deletion)

Some relevant references:

@jgomer2001

This comment has been minimized.

Collaborator

jgomer2001 commented Dec 23, 2017

value selection filters support added

@jgomer2001 jgomer2001 closed this Dec 23, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment