-
Notifications
You must be signed in to change notification settings - Fork 573
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
Cannot find name 'ReadableStream'. #3063
Comments
Hi @fishcharlie, thanks for reaching out. I am not able to reproduce this. Can you show how you set up your credentials and region configuration? |
@vudh1 I don't think credentials or region is relevant here. Both are just the default environment variables tho. Nothing passed into the object itself. Here you can find a link that will expire in 1 week that contains a zip file of a project used to reproduce this: https://fishcharliepersonalbucket.s3-us-west-2.amazonaws.com/tmp.zip?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAJ4K33MVV3APAWBLQ/20211127/us-west-2/s3/aws4_request&X-Amz-Date=20211127T001149Z&X-Amz-Expires=604800&X-Amz-SignedHeaders=host&X-Amz-Signature=e3cb3d78504e039905daa374bac615606a6f6e72b1d8defd6f414d717b705863. With that you should be able to run Or if you don't want to download that, you can do the following:
{
"compilerOptions": {
"module": "commonjs",
"moduleResolution": "node",
"lib": ["es2015"],
"target": "es2017",
"outDir": "dist",
"sourceMap": true,
"declaration": true,
"noImplicitThis": true,
"strict": false
},
"include": ["./lib/**/*"]
}
Hopefully this helps. Let me know if I can help any further. |
I had the same errors and a similar one from @aws-sdk/util-dynamodb: Was able to build after adding |
Adding |
This is a duplicate of #2896. |
I started encountering this after the changes from #3779 were pulled into my project. I can work around this by adding either
|
The other option is to use declaration merging to stub out the missing interface that aws-sdk assumes is available. Add this where you are exporting code that uses the SDK:
That should allow you to compile without hauling in the entire DOM lib. |
Good enough to keep it running |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread. |
Describe the bug
Cannot find name 'ReadableStream'.
Your environment
SDK version number
@aws-sdk/client-s3@3.42.0
Is the issue in the browser/Node.js/ReactNative?
Node.js
Details of the browser/Node.js/ReactNative version
Steps to reproduce
Observed behavior
Expected behavior
No TypeScript errors
Screenshots
Additional context
Also, when running this code, it doesn't print the contents, but instead a large object that looks like a response object. It should print the contents of the file.
The text was updated successfully, but these errors were encountered: