Skip to content

Support alternate patch operations without a path - #316

Merged
kqarryzada merged 2 commits into
masterfrom
DS-52017-allow-nested-patch
Oct 2, 2026
Merged

kqarryzada merged 2 commits into
masterfrom
DS-52017-allow-nested-patch

Conversation

@kqarryzada

Copy link
Copy Markdown
Collaborator

This commit adds support for patch operations with an empty "path" field and nested attribute targets within its "value" field. An example JSON has been added to the CHANGELOG. Although this form of patch operation is not defined by the standard, it is used by some SCIM provisioners. Applications that handle requests from these SCIM provisioners may now accept these requests and apply them to SCIM resource objects.

This update includes a new maximum array size for the target of a patch operation. The value is 20,000 by default, but may be modified with the "com.unboundid.scim2.common.utils.JsonUtils.maxPatchArraySize" property.

Reviewer: dougbulkley
Reviewer: vyhhuang

JiraIssue: DS-52017

Resolves #224

This commit adds support for patch operations with an empty "path" field
and nested attribute targets within its "value" field. An example JSON
has been added to the CHANGELOG. Although this form of patch operation
is not defined by the standard, it is used by some SCIM provisioners.
Applications that handle requests from these SCIM provisioners may now
accept these requests and apply them to SCIM resource objects.

This update includes a new maximum array size for the target of a patch
operation. The value is 20,000 by default, but may be modified with the
"com.unboundid.scim2.common.utils.JsonUtils.maxPatchArraySize" property.

Reviewer: dougbulkley
Reviewer: vyhhuang

JiraIssue: DS-52017
@kqarryzada kqarryzada self-assigned this Oct 2, 2026
@pingidentity pingidentity locked and limited conversation to collaborators Oct 2, 2026

@dougbulkley dougbulkley left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sent Khalid some comments over slack on this change.

@vyhhuang vyhhuang left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes look good to me

@kqarryzada
kqarryzada merged commit 6cf6195 into master Oct 2, 2026
6 checks passed
@kqarryzada
kqarryzada deleted the DS-52017-allow-nested-patch branch October 2, 2026 21:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Nested fields not patched correctly when path field omitted

3 participants