Skip to content

chore(checksums): add crc32c and sha1 checksums#8126

Merged
kuhe merged 1 commit into
mainfrom
kuhe/feat/crc32c
Jun 30, 2026
Merged

chore(checksums): add crc32c and sha1 checksums#8126
kuhe merged 1 commit into
mainfrom
kuhe/feat/crc32c

Conversation

@kuhe

@kuhe kuhe commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Issue

Related: #1

Description

  • Add crc32c and sha1 checksums locally
  • reorganize @aws-sdk/checksums into submodules for flex/sha/crc/md5

Testing

test vectors and CI, e2e tests

Checklist

  • If the PR is a feature, add integration tests (*.integ.spec.ts) or E2E tests.
    • It's not a feature.
  • My E2E tests are resilient to concurrent i/o.
    • I didn't write any E2E tests.
  • I added access level annotations e.g. @public, @internal tags and enabled doc generation on the package. Remember that access level annotations go below the description, not above.
    • I didn't add any public functions.
  • Streams - how do they work?? My WebStream readers/locks are properly lifecycled. Node.js stream backpressure is handled. Error handling.
    • No streams here.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@kuhe
kuhe force-pushed the kuhe/feat/crc32c branch 4 times, most recently from 98c37ec to 20d9080 Compare June 25, 2026 18:56
Comment thread packages-internal/checksums/src/submodules/sha/sha1/Sha1Js.ts Dismissed
Comment thread packages-internal/checksums/src/submodules/sha/sha1/Sha1Js.ts Dismissed
Comment thread packages-internal/checksums/src/submodules/sha/sha1/Sha1Js.ts Dismissed
@kuhe
kuhe force-pushed the kuhe/feat/crc32c branch from 20d9080 to 6e7f89a Compare June 29, 2026 16:31
@kuhe
kuhe marked this pull request as ready for review June 29, 2026 16:37
@kuhe
kuhe requested a review from a team as a code owner June 29, 2026 16:37
@kuhe
kuhe force-pushed the kuhe/feat/crc32c branch 5 times, most recently from 3880634 to 99b8aec Compare June 29, 2026 18:57
@kuhe
kuhe force-pushed the kuhe/feat/crc32c branch from 99b8aec to 8bd45df Compare June 29, 2026 18:59
@kuhe
kuhe merged commit ea8faa0 into main Jun 30, 2026
8 checks passed
@kuhe
kuhe deleted the kuhe/feat/crc32c branch June 30, 2026 14:24
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.

3 participants