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.

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

feat: improve support for fetch and web-streams in Node.js #1256

Merged
merged 7 commits into from
May 8, 2024

Conversation

kuhe
Copy link
Contributor

@kuhe kuhe commented May 1, 2024

Issue #, if available:
aws/aws-sdk-js-v3#4619

Description of changes:

  • streamCollector and sdkStreamMixin components that are part of the client config, specifically the SerdeContext subset, have been modified to allow web streams

packages/fetch-http-handler/README.md Outdated Show resolved Hide resolved
packages/node-http-handler/src/stream-collector/index.ts Outdated Show resolved Hide resolved
kuhe and others added 6 commits May 7, 2024 19:58
Co-authored-by: Trivikram Kamat <16024985+trivikr@users.noreply.github.com>
Co-authored-by: Trivikram Kamat <16024985+trivikr@users.noreply.github.com>
@kuhe kuhe changed the title feat: improve support for fetch/web-streams in Node.js feat: improve support for fetch and web-streams in Node.js May 7, 2024
@kuhe
Copy link
Contributor Author

kuhe commented May 8, 2024

I tested this by running our dynamodb e2e suite using fetch. S3 get/putObject also seem to work.

@kuhe kuhe merged commit 3500f34 into smithy-lang:main May 8, 2024
7 checks passed
@kuhe kuhe deleted the feat/fetch-in-node branch May 8, 2024 16:32
kuhe added a commit to kuhe/smithy-typescript that referenced this pull request May 28, 2024
…ng#1256)

* feat: improve fetch-http-handler compatibility in Node.js

* add changeset

* unit tests

* formatting

* Update packages/fetch-http-handler/README.md

Co-authored-by: Trivikram Kamat <16024985+trivikr@users.noreply.github.com>

* Update packages/node-http-handler/src/stream-collector/index.ts

Co-authored-by: Trivikram Kamat <16024985+trivikr@users.noreply.github.com>

* test: modify conditional unit tests

---------

Co-authored-by: Trivikram Kamat <16024985+trivikr@users.noreply.github.com>
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.

None yet

3 participants