-
Notifications
You must be signed in to change notification settings - Fork 74
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
Support ADD patch ops targeting existing values #230
Support ADD patch ops targeting existing values #230
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had one implementation concern, but all of my other remarks are comment based and don't necessarily require action.
scim2-sdk-common/src/main/java/com/unboundid/scim2/common/messages/PatchOperation.java
Outdated
Show resolved
Hide resolved
scim2-sdk-common/src/main/java/com/unboundid/scim2/common/messages/PatchOperation.java
Outdated
Show resolved
Hide resolved
scim2-sdk-common/src/main/java/com/unboundid/scim2/common/messages/PatchOperation.java
Outdated
Show resolved
Hide resolved
scim2-sdk-common/src/main/java/com/unboundid/scim2/common/messages/PatchOperation.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Khalid and I met over zoom yesterday to discuss this and my comments are in the DS issue.
8069300
to
a33e2a1
Compare
Added a release note that indicates that the Jackson dependency was upgraded.
In the 3.0.0 release, we implemented support for ADD patch operations with a value filter. The SDK would always append a new value to the array, since it is technically an "add". However, this does not play well with SCIM provisioners that send multiple individual updates, since these intend to target the same value within a multi-valued attribute, such as a user's work email. The intention is to add a new value to the field that matches the path filter (e.g., "type eq \"work\""). This commit adds support for this behavior. The new behavior is available via an opt-in setting in a static boolean variable, PatchOperation.APPEND_NEW_PATCH_VALUES_PROPERTY. To opt into this setting, set the value of this variable to false. As a result of this change, the version of the SCIM SDK has been updated to 3.2.0-SNAPSHOT. Reviewer: vyhhuang Reviewer: dougbulkley JiraIssue: DS-49194
a33e2a1
to
6f74c28
Compare
In the 3.0.0 release, we implemented support for ADD patch operations
with a value filter. The SDK would always append a new value to the
array, since it is technically an "add". However, this does not play
well with SCIM provisioners that send multiple individual updates,
since these intend to target the same value within a multi-valued
attribute, such as a user's work email. The intention is to add a new
value to the field that matches the path filter (e.g.,
"type eq "work"").
The new behavior is available via an opt-in setting in a static boolean
variable, PatchOperation.APPEND_NEW_PATCH_VALUES_PROPERTY. To opt into
this setting, set the value of this variable to false.
As a result of this change, the version of the SCIM SDK has been updated
to 3.2.0-SNAPSHOT.
Reviewer: vyhhuang
Reviewer: dougbulkley
JiraIssue: DS-49194
Resolves #213