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

The 'Request' library is deprecated #22

Closed
antony opened this issue Feb 8, 2021 · 12 comments
Closed

The 'Request' library is deprecated #22

antony opened this issue Feb 8, 2021 · 12 comments

Comments

@antony
Copy link

antony commented Feb 8, 2021

See https://github.com/request/request and request/request#3142

This library should probably be using node-fetch

@otisg
Copy link
Member

otisg commented Feb 8, 2021

Thanks, @antony. Any chance you could submit a PR?

@antony
Copy link
Author

antony commented Feb 8, 2021

@otisg I might be able to yeah - will do as soon as I'm able to.

@gsf4726
Copy link

gsf4726 commented Mar 20, 2023

This is now more critical as a vulnerability (CVE-2023-28155) was published recently, so it's now tripping npm audit.

@gsf4726
Copy link

gsf4726 commented Mar 23, 2023

@otisg is it on Sematext's roadmap to fix the CVE-2023-28155 vulnerability in this module?

@otisg
Copy link
Member

otisg commented Mar 23, 2023

@gsf4726 Not planned currently, unless we get a PR. Don't think this module has (m)any users.

@antony
Copy link
Author

antony commented Mar 23, 2023

https://www.npmjs.com/package/logsene-js

3064 weekly downloads, and a critical security vulnerability?

not sure I agree here.

I'd love to submit a PR but I too am pressed for time so haven't managed to yet.

@otisg
Copy link
Member

otisg commented Mar 23, 2023

@antony Oh I don't believe those stats. I suspect 99% of those npm stats numbers are from bots/automated downloads/updates.

@antony
Copy link
Author

antony commented Mar 23, 2023

automated downloads/updates meaning CI? that's usage. I'm not sure what bots download npm dependencies, otherwise. I certainly haven't seen any evidence of that. I would say that the library has a reasonable amount of usage.

@yelworc
Copy link

yelworc commented Mar 28, 2023

@gsf4726 Not planned currently, unless we get a PR. Don't think this module has (m)any users.

Hmm, that sounds vaguely concerning. Is this not the recommended package for shipping logs to Logsene in a Node.js app? The name kinda sounds like it is 😆

I'm definitely using this package in a production API – if it isn't actually being maintained, I need to look for alternatives. That's fine, but it should be clearly and visibly communicated (eg. by archiving the repo, deprecating the npm package, etc).

@otisg
Copy link
Member

otisg commented Mar 29, 2023

Ugh, sorry folks, my mistake - I was thinking about https://github.com/sematext/logsene-cli
So, yes, we will address this CVE issue.

@regiluze
Copy link
Contributor

@antony @yelworc we've just released a new version of the library removing all vulnerabilities. https://github.com/sematext/logsene-js/releases/tag/1.1.76

@yelworc
Copy link

yelworc commented Apr 21, 2023

Thank you, much appreciated!

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

5 participants