Skip to content

Use ObjectInputFilter for deserialization (#5469) - #5572

Merged
Swiddis merged 5 commits into
opensearch-project:2.19from
ritvibhatt:filter-deserialization-backport
Jun 24, 2026
Merged

Use ObjectInputFilter for deserialization (#5469)#5572
Swiddis merged 5 commits into
opensearch-project:2.19from
ritvibhatt:filter-deserialization-backport

Conversation

@ritvibhatt

Copy link
Copy Markdown
Contributor

Description

Adds ObjectInputFilter for deserialization in PlanSerializer, DefaultExpressionSerializer, and RelJsonSerializer by implementing a allowlist-based approach that only allows specific classes.

Related Issues

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • New functionality has javadoc added.
  • New functionality has a user manual doc added.
  • New PPL command checklist all confirmed.
  • API changes companion pull request created.
  • Commits are signed per the DCO using --signoff or -s.
  • Public documentation issue/PR created.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

ritvibhatt and others added 2 commits June 19, 2026 14:16
* use ObjectInputFilter for deserialization for cursors and script pushdown

Signed-off-by: Ritvi Bhatt <ribhatt@amazon.com>

* update allowlist

Signed-off-by: Ritvi Bhatt <ribhatt@amazon.com>

---------

Signed-off-by: Ritvi Bhatt <ribhatt@amazon.com>
Signed-off-by: Ritvi Bhatt <ribhatt@amazon.com>
@ritvibhatt
ritvibhatt marked this pull request as ready for review June 22, 2026 22:21
Signed-off-by: Ritvi Bhatt <ribhatt@amazon.com>
Signed-off-by: Ritvi Bhatt <ribhatt@amazon.com>
Signed-off-by: Ritvi Bhatt <ribhatt@amazon.com>
@Swiddis
Swiddis merged commit 9ab5038 into opensearch-project:2.19 Jun 24, 2026
35 of 38 checks passed
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

Successfully merging this pull request may close these issues.

2 participants