-
Notifications
You must be signed in to change notification settings - Fork 29
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
ReferenceError: fetch is not defined #7
Comments
@PAUL-TX This seems like an issue with the Cognito JS SDK. Can I see your |
Hi Jay, I’m not sure which project I was doing this for. One thing I know though is that I’m fastidious about always using the latest version of packages.
From: Jay V
Sent: Friday, February 16, 2018 4:06 PM
To: AnomalyInnovations/aws-api-gateway-cli-test
Cc: paul-tx; Mention
Subject: Re: [AnomalyInnovations/aws-api-gateway-cli-test] ReferenceError:fetch is not defined (#7)
@PAUL-TX This seems like an issue with the Cognito JS SDK. Can I see your package.json?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
…---
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus
|
I encountered a similar issue in another script I was building using amazon-cognito-identity-js library v2.0.0 on node v6.10.3 I was able to fix it by adding node-fetch npm to the project and setting a global for it. |
@sw-waterford Did you add this to the |
This was in a different project/library if it was added to the index of this project it should be a suitable replacement for the WindowMock library. I have not encountered the issue with this module so I have not tried swapping them out. |
I'm following the tutorial at: https://serverless-stack.com/chapters/test-the-apis.html
I'm using node v8.9.4 and the latest packages as of 2/12/2018
I get this error:
The text was updated successfully, but these errors were encountered: