Skip to content
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

[Client encryption]: Adds JsonNode support on deserialization path #4787

Merged

Conversation

JanHyka
Copy link
Contributor

@JanHyka JanHyka commented Oct 10, 2024

Pull Request Template

Description

  • Added support for System.Text.JsonNode DOM on deserializer and decryption path
  • Drops custom byte[] converter as JsonNode can take Memory directly

To be processed after #4780

Type of change

Please delete options that are not relevant.

  • [] New feature (non-breaking change which adds functionality)
  • [] This change requires a documentation update

Closing issues

Contributes to #4678

juraj-blazek and others added 30 commits September 12, 2024 13:41
~ reduce validations overhead
~ update api file
- drop unnecessary classes
~ Microsoft.Azure.Cosmos.Encryption.Custom now references System.Text.Json 8.0.5 as per security advisory
@kirankumarkolli
Copy link
Member

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@JanHyka JanHyka changed the title [Client encryption]: Add JsonNode support on deserialization path [Client encryption]: Adds JsonNode support on deserialization path Oct 16, 2024
@kirankumarkolli kirankumarkolli merged commit f88605c into Azure:master Oct 16, 2024
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

4 participants