Skip to content
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

Question: Any plans for "request" migration? #328

Open
sseide opened this issue Aug 24, 2021 · 7 comments
Open

Question: Any plans for "request" migration? #328

sseide opened this issue Aug 24, 2021 · 7 comments

Comments

@sseide
Copy link
Contributor

sseide commented Aug 24, 2021

Description/Steps to reproduce

I just want to ask if there are any plans already to move from the abandoned "request" library to another maintained library.
(https://github.com/request/request). As this is deprecated since Februar last year is is just a question of time until higher priority problems arise with this lib.

By now there is only one low prio thing with old sha1 hashes in oauth (request/request#3385) but it will probably not be the last one...

Is there a decision which new library shall be used? A possible comparison of different libraries use compile here:
https://github.com/sindresorhus/got#comparison

Link to reproduction sandbox

Expected result

Additional information

@dhmlau
Copy link
Member

dhmlau commented Aug 25, 2021

@sseide, thanks for the reference. We've used axios for some (maybe one) of the loopback-next packages but I think @marioestradarosa was having some issues with that.

@marioestradarosa, any insights?

@stale stale bot added the stale label Oct 25, 2021
@loopbackio loopbackio deleted a comment from stale bot Oct 25, 2021
@stale stale bot removed the stale label Oct 25, 2021
@sseide
Copy link
Contributor Author

sseide commented Nov 15, 2021

@marioestradarosa any information or thoughts about this?

And as expected - some more things coming up in "request" dependencies that will not be fixed as they need updates to "request" itself to allow newer versions of the libraries. Therefor it is safe to assume CVE-2021-3918 within "json-schema" (with score high) as an indirect dependency will not be fixed in "request" because request needs an updated "http-signature" library...

And these things start to be reported in automated security scanners, will probably generate a lot of requests about request in the forseeable future.

@s100
Copy link
Contributor

s100 commented May 15, 2023

request now has a significant CVE in it which will not be fixed.

Please note that strong-soap also makes use of request indirectly, via httpntlm-maa. I have raised an issue against httpntlm-maa, but the package has not been maintained for over three years, and I do not expect it to be fixed. So, please could you also consider stopping using httpntlm-maa or migrating away from it to something else.

@sseide
Copy link
Contributor Author

sseide commented May 15, 2023

@s100 httpntlm-maa has no direct dependency to request - its only declared as a peer-dependency of either request or node-fetch. IT does not install these by itself.

Therefor it is up to the user which one he uses... But as strong-soap requires request too, this one is used.
As soon as strong-soap might switch to node-fetch dropping request everything is fine - even for httpntlm-maa

@s100
Copy link
Contributor

s100 commented May 15, 2023

@s100 httpntlm-maa has no direct dependency to request - its only declared as a peer-dependency of either request or node-fetch. IT does not install these by itself.

This was true at one point but this is no longer the case. Starting from npm 7 (which is part of Node.js 15+, October 2020), peer dependencies such as request are installed by default.

@sseide
Copy link
Contributor Author

sseide commented May 15, 2023

Oh - thanks for the reply. Did not realize this as these packages where direct dependencies here too on our side...

To bad, nothing to show what might be ok without installing it right away, and "optionalDependencies" are installed to per default, not installing them must be requested explicit too (same as with peer now) ... :-/
Do you now another solution except not mentioning these dependencies at all?

@s100
Copy link
Contributor

s100 commented May 15, 2023

There are plenty of alternative HTTP packages to request which you could use to get equivalent functionality. I like node-fetch, personally. Or, as this is software intended to run in Node.js, you could use the built-in http package directly.

I don't know what alternatives there are for httpntlm-maa - I don't know what this package is or does.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants