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

Consider updating version of "eslint-import-resolver-typescript" in "eslint-config-next" #35806

Closed
1 task done
DavideBecker opened this issue Apr 1, 2022 · 2 comments
Closed
1 task done
Labels
bug Issue was opened via the bug report template. Linting Related to `next lint` or ESLint with Next.js.

Comments

@DavideBecker
Copy link

Verify canary release

  • I verified that the issue exists in Next.js canary release

Provide environment information

Operating System:
  Platform: darwin
  Arch: arm64
  Version: Darwin Kernel Version 21.1.0: Wed Oct 13 17:33:01 PDT 2021; root:xnu-8019.41.5~1/RELEASE_ARM64_T6000
Binaries:
  Node: 14.18.2
  npm: 6.14.15
  Yarn: 1.22.17
  pnpm: N/A
Relevant packages:
  next: 12.1.4
  react: 17.0.2
  react-dom: 17.0.2

What browser are you using? (if relevant)

No response

How are you deploying your application? (if relevant)

No response

Describe the Bug

The minimist package has a security vulnerability in versions <=1.2.5.

npm audit reports the following dependency path:

eslint-config-next > eslint-import-resolver-typescript > tsconfig-paths > json5 > minimist

All other packages and their dependencies have updated to a non-vulnerable version of minimist. However, npm audit still reports a vulnerability since eslint-config-next depends on v2.4.0 of eslint-import-resolver-typescript. The dependency has been updated in v2.6.0 (See: import-js/eslint-import-resolver-typescript#98)

Expected Behavior

Update the eslint-import-resolver-typescript dependency in

"eslint-import-resolver-typescript": "2.4.0",
from v2.4.0 to v2.6.0

To Reproduce

Install eslint-config-next and run npm audit

@DavideBecker DavideBecker added the bug Issue was opened via the bug report template. label Apr 1, 2022
@balazsorban44 balazsorban44 added the Linting Related to `next lint` or ESLint with Next.js. label Apr 1, 2022
@ijjk
Copy link
Member

ijjk commented Apr 28, 2022

Closing as this should be resolved by #35781

@ijjk ijjk closed this as completed Apr 28, 2022
@github-actions
Copy link
Contributor

This closed issue has been automatically locked because it had no new activity for a month. If you are running into a similar issue, please create a new issue with the steps to reproduce. Thank you.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 29, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue was opened via the bug report template. Linting Related to `next lint` or ESLint with Next.js.
Projects
None yet
Development

No branches or pull requests

3 participants