-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Examples use request
library, which has been deprecated.
#49
Comments
Can't believe it has been 2 years and no one has considered updating this. |
I'd also hope to see this get updated. |
I just made a pull request for this issue: @scowalt @terryjiang2020 I would appreciate if either of you could take a look at it to check if it looks good. |
@zhenyu0519 Why was your pull request for this closed? |
@perploug I'm not sure if anyone is really assigned to maintain this repo internally at Spotify anymore, but saw that you made a commit recently. Could you merge my pull request or find someone who could? |
The
request
library is use in these examples to perform network requests from a node server instance.This library has been deprecated.
This post suggests some alternative libraries to use. I personally would recommend
node-fetch
, since it matches the knownfetch
API from browsers and uses promises.The text was updated successfully, but these errors were encountered: