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.

Sign up for GitHub

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

Jump to bottom

CRC32C implementation compatible with HashAlgorithm without requiring a CRT dependency #3611

Open
2 tasks
leonnis12 opened this issue Jan 17, 2025 · 1 comment
Open
2 tasks

CRC32C implementation compatible with HashAlgorithm without requiring a CRT dependency #3611

leonnis12 opened this issue Jan 17, 2025 · 1 comment
Labels
feature-request A feature should be added or improved. module/sdk-core needs-review p2 This is a standard priority issue

Comments

@leonnis12
Copy link

leonnis12 commented Jan 17, 2025 •
edited
Loading

Describe the feature

Recently there was a change to refactor the CRC32 checksum implementation to be compatible with HashAlgorithm without requiring a CRT dependency. It would be appropriate to do the same for CRC32C to not have the CRT dependency for either of them.

Use Case

Not requiring the CRT dependency for any of the checksums.
I'm also facing some incompatibility issues with the CRT dependency on Alpine based docker images, which happens now only for CRC32C, and would be not be an issue after this change.

Proposed Solution

No response

Other Information

No response

Acknowledgements

  • I may be able to implement this feature request
  • This feature might incur a breaking change

AWS .NET SDK and/or Package version used

AWS .NET SDK 3.7.963.0

Targeted .NET Platform

.NET 8.0

Operating System and version

Windows 11, Alpine
@leonnis12 leonnis12 added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels Jan 17, 2025
@ashishdhingra ashishdhingra added investigating This issue is being investigated and/or work is in progress to resolve the issue. module/sdk-core p2 This is a standard priority issue and removed needs-triage This issue or PR still needs to be triaged. labels Jan 17, 2025
@ashishdhingra
Copy link
Contributor

ashishdhingra commented Jan 17, 2025 •
edited
Loading

@leonnis12 Good morning. Thanks for opening the issue. This needs review with the team.

@ashishdhingra ashishdhingra added needs-review and removed investigating This issue is being investigated and/or work is in progress to resolve the issue. labels Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature-request A feature should be added or improved. module/sdk-core needs-review p2 This is a standard priority issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@leonnis12 @ashishdhingra