Skip to content

Add CompositeMLDsa CNG (NCrypt)#130053

Open
PranavSenthilnathan wants to merge 7 commits into
dotnet:mainfrom
PranavSenthilnathan:cmldsa-ncrypt
Open

Add CompositeMLDsa CNG (NCrypt)#130053
PranavSenthilnathan wants to merge 7 commits into
dotnet:mainfrom
PranavSenthilnathan:cmldsa-ncrypt

Conversation

@PranavSenthilnathan

@PranavSenthilnathan PranavSenthilnathan commented Jun 30, 2026

Copy link
Copy Markdown
Member

Wires up Composite ML-DSA CNG support.

The APIs in #130052 is pending approval so they're internal in this PR.

PKCS8 export does not work in Windows for Composite ML-DSA (yet), so encrypted export only keys can't currently be exported. Also, PKCS8 export is done by exporting the private key instead.

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @bartonjs, @vcsjones, @dotnet/area-system-security
See info in area-owners.md if you want to be subscribed.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 19 out of 19 changed files in this pull request and generated 2 comments.

Comment thread src/libraries/System.Security.Cryptography/ref/System.Security.Cryptography.cs Outdated
Copilot AI review requested due to automatic review settings July 7, 2026 08:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 20 out of 20 changed files in this pull request and generated 2 comments.

Copilot AI review requested due to automatic review settings July 13, 2026 21:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 20 out of 20 changed files in this pull request and generated 3 comments.

Comment thread src/libraries/System.Security.Cryptography/src/Resources/Strings.resx Outdated
Comment thread src/libraries/Microsoft.Bcl.Cryptography/src/Resources/Strings.resx Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 13, 2026 21:49

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 20 out of 20 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings July 14, 2026 17:08
@PranavSenthilnathan PranavSenthilnathan enabled auto-merge (squash) July 14, 2026 17:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 20 out of 20 changed files in this pull request and generated no new comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants