Replacing deprecated request dependency

loopbackio / loopback-connector-rest

Connect Loopback to a REST API

http://loopback.io/doc/en/lb2/REST-connector.html

Other

75 stars 81 forks source link

Replacing deprecated request dependency #164

Open fidgi opened 2 years ago

fidgi commented 2 years ago

Suggestion

Replacing the deprecated request package by a modern equivalent like axios or superagent.

More alternatives mentionned here

Use Cases

N/A

Examples

N/A

Acceptance criteria

TBD - will be filled by the team.

shubhamp-sf commented 1 year ago

Already being discussed here: https://github.com/loopbackio/loopback-connector-rest/issues/147

samarpanB commented 6 months ago

In order to introduce a new library in place of request, it will need a full rewrite of the codebase. Also, there are a few alternatives available like axios, got or fetch (natively available since nodejs v21). I think we need to see what could be our best and future proof alternative. I would propose to move to native fetch module since that's going to be native to nodejs in coming future. Thoughts @raymondfeng @dhmlau @achrinza ?