Skip to content

feat: default encryptAsync to v2#8936

Merged
bdesoky merged 1 commit into
masterfrom
WCN-723
Jun 3, 2026
Merged

feat: default encryptAsync to v2#8936
bdesoky merged 1 commit into
masterfrom
WCN-723

Conversation

@bdesoky
Copy link
Copy Markdown
Contributor

@bdesoky bdesoky commented Jun 3, 2026

Ticket: WCN-723

This pull request updates the encryption in encryptAsync to default to v2

Encryption logic improvements:

  • Changed the default encryption version in encryptAsync to v2 (Argon2id + AES-256-GCM), so unless explicitly set to v1, new data will use the more secure encryption scheme.

Testing enhancements:

  • Updated unit tests to verify that v2 is the default and explicit v1 encryption still works.

@bdesoky bdesoky requested review from a team as code owners June 3, 2026 19:09
@bdesoky bdesoky requested review from Marzooqa and s84krish June 3, 2026 19:09
@linear-code
Copy link
Copy Markdown

linear-code Bot commented Jun 3, 2026

WCN-723

@bdesoky
Copy link
Copy Markdown
Contributor Author

bdesoky commented Jun 3, 2026

Synced with @pranavjain97, as this change doesnt remove the encrypt() function and no external callers are using encryptAsync, this change is not breaking. decryptAsync can handle v1/2 decryption with no changes

@bdesoky bdesoky merged commit b46ef91 into master Jun 3, 2026
33 of 34 checks passed
@bdesoky bdesoky changed the title feat: default encryptAsync to v2 and add whitespace normalization feat: default encryptAsync to v2 Jun 3, 2026
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