Skip to content

Comments

Add support for SHA-384 and SHA-512 hash algorithms#82452

Open
jcouv wants to merge 2 commits intodotnet:mainfrom
jcouv:new-hash
Open

Add support for SHA-384 and SHA-512 hash algorithms#82452
jcouv wants to merge 2 commits intodotnet:mainfrom
jcouv:new-hash

Conversation

@jcouv
Copy link
Member

@jcouv jcouv commented Feb 19, 2026

@jcouv jcouv self-assigned this Feb 19, 2026
@dotnet-policy-service dotnet-policy-service bot added the Needs API Review Needs to be reviewed by the API review council label Feb 19, 2026
@dotnet-policy-service
Copy link
Contributor

This PR modifies public API files. Please follow the instructions at https://github.com/dotnet/roslyn/blob/main/docs/contributing/API%20Review%20Process.md for ensuring all public APIs are reviewed before merging.

var compilation = CreateCompilation(new[] { tree1, tree256, tree384, tree512 });

// The checksumAlgorithm values for SHA-384 and SHA-512 below use raw GUIDs instead of friendly
// names (like "SHA384"/"SHA512") because SRM's PdbToXmlConverter doesn't yet recognize these GUIDs.
Copy link
Member

@tmat tmat Feb 19, 2026

Choose a reason for hiding this comment

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

This will fix it: dotnet/symreader-converter#326. Let's wait for the new package, should be available shortly. #Closed

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Updated. Thanks

@jcouv jcouv marked this pull request as ready for review February 20, 2026 00:57
@jcouv jcouv requested review from a team as code owners February 20, 2026 00:57
@@ -1,4 +1,6 @@
Microsoft.CodeAnalysis.Emit.EmitDifferenceOptions.MethodImplEntriesSupported.get -> bool
Microsoft.CodeAnalysis.Text.SourceHashAlgorithm.Sha384 = 3 -> Microsoft.CodeAnalysis.Text.SourceHashAlgorithm
Microsoft.CodeAnalysis.Text.SourceHashAlgorithm.Sha512 = 4 -> Microsoft.CodeAnalysis.Text.SourceHashAlgorithm
Copy link
Member

Choose a reason for hiding this comment

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

Do we need public API review?

Copy link
Member Author

Choose a reason for hiding this comment

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

It will. Talked to Fred, we'll do it by email since trivial.

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

Labels

Area-Compilers Needs API Review Needs to be reviewed by the API review council

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants