Skip to content

Making toDecrypt and toEncrypt optional params - #115

Merged
rfeelin merged 2 commits into
Mastercard:mainfrom
rfeelin:feature/make-toEncrypt-toDecrypt-optional
Nov 11, 2026
Merged

rfeelin merged 2 commits into
Mastercard:mainfrom
rfeelin:feature/make-toEncrypt-toDecrypt-optional

Conversation

@rfeelin

@rfeelin rfeelin commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

PR checklist

  • [ x] An issue/feature request has been created for this PR (internal)
  • [ x] Pull Request title clearly describes the work in the pull request and the Pull Request description provides details about how to validate the work. Missing information here may result in a delayed response.
  • [ x] File the PR against the master branch
  • [ x] The code in this PR is covered by unit tests

Description

Making the toDecrypt and toEncrypt optional params so both don't have to be set at the same time to use the libraries. Previously something like this wouldn't work

			"paths": [
				{
					"path": "/adoptions",
					"toDecrypt": [
						{
							"element": "$.encryptedOwner",
							"obj": "owner"
						},
						{
							"element": "$.adoption.encryptedOwner",
							"obj": "owner"
						}
					]
				}
			],

Co-authored-by: joseph-neeraj <joseph.neeraj@gmail.com>
@rfeelin
rfeelin merged commit f5ebcae into Mastercard:main Nov 11, 2026
7 of 8 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.

3 participants