Roadmap to migrate away from request

[DanielRuf]

Summary

request is in maintenance mode now so we should check what we have to do to migrate away from it.

Relevant information

request/request#3142

[KyleAMathews]

There's no specific rush to do this as request isn't going to stop working :-)

I did a quick look and we only use request once in our code base — 

gatsby/packages/gatsby/src/commands/develop.js

Line 193 in a43288d

request(proxiedUrl).on(`error`, err => {

If someone wants to PR moving to a different library, that seems reasonable.

[DanielRuf]

I did a quick look and we only use request once in our code base

I thought there were more places where we still use request.

There's no specific rush to do this as request isn't going to stop working :-)

Sure but let's keep an eye on this.

If someone wants to PR moving to a different library, that seems reasonable.

👍

[sindresorhus]

If you're willing to give got a try, I think you could simply replace request( with got.stream( in the referenced code.

See: https://github.com/sindresorhus/got/blob/master/migration-guides.md#more-about-streams

[wardpeet]

sweet! we're open for PRs but this isn't really urgent.

[ryandrew14]

Looked into it a little more, and I think @sindresorhus is right -- got seems like the best option. They have a great comparison on their website of how it stacks up against other Node libraries for making requests. I will make a PR for this tonight.

[pieh]

Just note on got - we still do support node 6 :( and got@^9 seems to require node: ">=8.6", so if we would switch we would need to use got@<=8 (which works on node: ">=4").

The node 6 support is super problematic for us, but we can't drop it in the middle of gatsby@2 (even if node 6 end of life is generally coming pretty soon - I think end of April 2019).

[ryandrew14]

Closed by #13434.

[sidharthachatterjee]

Published in gatsby@2.3.25

[armchairdeity]

If you're willing to give got a try, I think you could simply replace request( with got.stream( in the referenced code.

See: https://github.com/sindresorhus/got/blob/master/migration-guides.md#more-about-streams

Broken link... :(

[DanielRuf]

@armchairdeity see https://github.com/sindresorhus/got/blob/master/documentation/migration-guides.md

[armchairdeity]

@armchairdeity see https://github.com/sindresorhus/got/blob/master/documentation/migration-guides.md

Thanks! 👍🏻

[tomconder]

@armchairdeity see https://github.com/sindresorhus/got/blob/master/documentation/migration-guides.md

They renamed the master branch to main. Now the url is: https://github.com/sindresorhus/got/blob/main/documentation/migration-guides/request.md