Skip to content

feat(middleware-flexible-checksums): allow custom checksums to be used in responses#7849

Merged
kuhe merged 1 commit intomainfrom
kuhe/feat/checksum
Mar 16, 2026
Merged

feat(middleware-flexible-checksums): allow custom checksums to be used in responses#7849
kuhe merged 1 commit intomainfrom
kuhe/feat/checksum

Conversation

@kuhe
Copy link
Copy Markdown
Contributor

@kuhe kuhe commented Mar 13, 2026

Issue

follows #7746

Description

allows novel checksum usage for responses

Testing

new integration 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 marked this pull request as ready for review March 16, 2026 14:55
@kuhe kuhe requested a review from a team as a code owner March 16, 2026 14:55
@kuhe kuhe force-pushed the kuhe/feat/checksum branch 2 times, most recently from 66e1587 to ca544f0 Compare March 16, 2026 15:23
@kuhe kuhe force-pushed the kuhe/feat/checksum branch from ca544f0 to f0584c7 Compare March 16, 2026 15:29
@kuhe kuhe merged commit 213defa into main Mar 16, 2026
7 checks passed
@kuhe kuhe deleted the kuhe/feat/checksum branch March 16, 2026 16:39
Copy link
Copy Markdown
Member

@trivikr trivikr left a comment

Choose a reason for hiding this comment

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

LGTM

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