Skip to content

fix(data-mask): compact JSON array when removing an element - #13818

Merged
AlinsRan merged 3 commits into
apache:masterfrom
Arjen10:fix/data-mask-nil
Aug 14, 2026
Merged

fix(data-mask): compact JSON array when removing an element#13818
AlinsRan merged 3 commits into
apache:masterfrom
Arjen10:fix/data-mask-nil

Conversation

@Arjen10

@Arjen10 Arjen10 commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Description

fix the deletion error

Which issue(s) this PR fixes:

Fixes #13817

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

@dosubot dosubot Bot added size:M This PR changes 30-99 lines, ignoring generated files. bug Something isn't working labels Aug 13, 2026
Comment thread apisix/plugins/data-mask.lua
@dosubot dosubot Bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Aug 14, 2026
Comment thread t/plugin/data-mask.t

@membphis membphis left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@AlinsRan
AlinsRan merged commit 48fd32b into apache:master Aug 14, 2026
16 checks passed
@Arjen10
Arjen10 deleted the fix/data-mask-nil branch August 14, 2026 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: data mask plugin failed to remove array element

4 participants