Duplicate code reduction part 2: clean up checksum classes

[zoewangg]

Motivation and Context

Clean up duplicate checksum logic in legacy checksum classes ("auth") and new checksum classes ("checksums"):

• Internal APIs: CrtBasedChecksumProvider, SdkCrc32 and SdkCrc32C
  • Remove duplicate checksumming logic that is in internal packages:
• Protected APIs:*Checksum classes such as Sha1Checksum:
  • we can't remove them because even though they are marked as private APIs, they are not in "internal" package properly. I made the change to remove the logic and reuse the new checksum logic

[sonarqubecloud]

Quality Gate passed

Issues
31 New issues
0 Accepted issues

Measures
0 Security Hotspots
87.3% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[github-actions]

This pull request has been closed and the conversation has been locked. Comments on closed PRs are hard for our team to see. If you need more assistance, please open a new issue that references this one.