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

[Bug] nyc-test-coverage uses unmaintained istanbul-instrumenter-loader #911

Closed
joebowbeer opened this issue Oct 10, 2022 · 1 comment · Fixed by #1000
Closed

[Bug] nyc-test-coverage uses unmaintained istanbul-instrumenter-loader #911

joebowbeer opened this issue Oct 10, 2022 · 1 comment · Fixed by #1000
Labels
bug Something isn't working

Comments

@joebowbeer
Copy link
Contributor

What are you really trying to do?

nyc-test-coverage uses unmaintained (archived) istanbul-instrumenter-loader, which results in npm audit findings

Describe the bug

Installing prints several deprecation warnings:

npm WARN deprecated urix@0.1.0: Please see https://github.com/lydell/urix#deprecated
npm WARN deprecated source-map-resolve@0.5.3: See https://github.com/lydell/source-map-resolve#deprecated
npm WARN deprecated source-map-url@0.4.1: See https://github.com/lydell/source-map-url#deprecated
npm WARN deprecated resolve-url@0.2.1: https://github.com/lydell/resolve-url#deprecated
npm WARN deprecated querystring@0.2.0: The querystring API is considered Legacy. new code should use the URLSearchParams API instead.
npm WARN deprecated chokidar@2.1.8: Chokidar 2 does not receive security updates since 2019. Upgrade to chokidar 3 with 15x fewer dependencies
npm WARN deprecated core-js@2.6.12: core-js@<3.23.3 is no longer maintained and not recommended for usage due to the number of issues. Because of the V8 engine whims, feature detection in old core-js versions could cause a slowdown up to 100x even if nothing is polyfilled. Some versions have web compatibility issues. Please, upgrade your dependencies to the actual version of core-js.
# npm audit report

ajv  <6.12.3
Severity: moderate
Prototype Pollution in Ajv - https://github.com/advisories/GHSA-v88g-cgmw-v5xw
fix available via `npm audit fix --force`
Will install @temporalio/nyc-test-coverage@1.2.0, which is a breaking change
node_modules/@temporalio/nyc-test-coverage/node_modules/schema-utils/node_modules/ajv
  schema-utils  <=0.4.3
  Depends on vulnerable versions of ajv
  node_modules/@temporalio/nyc-test-coverage/node_modules/schema-utils
    istanbul-instrumenter-loader  >=3.0.0-beta.0
    Depends on vulnerable versions of schema-utils
    node_modules/@temporalio/nyc-test-coverage/node_modules/istanbul-instrumenter-loader
      @temporalio/nyc-test-coverage  >=1.3.0
      Depends on vulnerable versions of istanbul-instrumenter-loader
      node_modules/@temporalio/nyc-test-coverage

glob-parent  <5.1.2
Severity: high
glob-parent before 5.1.2 vulnerable to Regular Expression Denial of Service in enclosure regex - https://github.com/advisories/GHSA-ww39-953v-wcq6
fix available via `npm audit fix`
node_modules/watchpack-chokidar2/node_modules/glob-parent
  chokidar  1.0.0-rc1 - 2.1.8
  Depends on vulnerable versions of glob-parent
  node_modules/watchpack-chokidar2/node_modules/chokidar
    watchpack-chokidar2  *
    Depends on vulnerable versions of chokidar
    node_modules/watchpack-chokidar2
      watchpack  1.7.2 - 1.7.5
      Depends on vulnerable versions of watchpack-chokidar2
      node_modules/@temporalio/nyc-test-coverage/node_modules/watchpack
        webpack  4.44.0 - 4.46.0
        Depends on vulnerable versions of watchpack
        node_modules/@temporalio/nyc-test-coverage/node_modules/webpack

9 vulnerabilities (4 moderate, 5 high)

Minimal Reproduction

  1. https://gitpod.io#https://github.com/joebowbeer/temporal-pendulum/tree/patch-client-workflow
  2. cd position-node
  3. npm ci
  4. npm audit

Additional context

See #872

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants