Skip to content

Remove timeout option and update node-fetch options #425

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

Merged
merged 2 commits into from
Apr 19, 2021

Conversation

nikithauc
Copy link
Contributor

Fixes #423.

The timeout option is removed from node-fetch options and the recommended way is cancel the request on timeout https://github.com/microsoftgraph/msgraph-sdk-javascript/blob/3.0.0/docs/CancellingAHTTPRequest.md.

@nikithauc nikithauc requested review from zengin, baywet and ddyett April 16, 2021 22:42
@nikithauc nikithauc added this to the 3.0.0 milestone Apr 16, 2021
@nikithauc nikithauc merged commit 0043f10 into 3.0.0 Apr 19, 2021
@nikithauc nikithauc linked an issue Apr 19, 2021 that may be closed by this pull request
2 tasks
@nikithauc nikithauc deleted the nikithauc/node-fetch-opts-update branch April 26, 2021 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

the timeout property of the fetchOptions is not working?
2 participants