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

Dependency "cb" causing security issues in Nexus because of missing LICENSE file and being 10 years from last update #423

Closed
awacode21 opened this issue Nov 28, 2022 · 5 comments · Fixed by #424

Comments

@awacode21
Copy link

Describe the problem

We are using express-openid-connect for our project to do the whole auth0 process.
But our production build is no longer able to build because Nexus complains about Security vulnaribilities. It is complaing about the dependency "cb" used by express-openid-connect as it is missing a LICENSE file and that the package is older than 5 years, actually the last update was 10 years ago.

Is there any chance to replace this outdated dependency?

What was the expected behavior?

My project should be able to use express-openid-connect without running into security vulnerabilities. I expect express-openid-connect to use mantained packages and not totally outdated stuff.

  • Version of this library used:
    LATEST
@kmannislands
Copy link
Contributor

Looks like the cb dependency is used in 2 files with 3 total calls. It is used to ensure a callback is executed once.

This is trivial to replace with code in express-openid-connect and a seems to me to be an example of the 'install a module for everything' anti pattern so commonplace the js ecosystem.

IMO, this dependency should definitely be removed.

adamjmcgrath pushed a commit that referenced this issue Nov 29, 2022
@awacode21
Copy link
Author

awacode21 commented Dec 2, 2022

thank you so much for the quick reply and fix. When the fix will be released? @adamjmcgrath @kmannislands

@adamjmcgrath
Copy link
Contributor

Hey @awacode21 - I'll put out a release next week

@awacode21
Copy link
Author

Great thanks!

@adamjmcgrath
Copy link
Contributor

👋 @awacode21 - this got released in 2.11.0

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

Successfully merging a pull request may close this issue.

3 participants