Skip to content
This repository has been archived by the owner on Jun 21, 2023. It is now read-only.

🚨 [security] Update codecov: 3.6.5 → 3.8.2 (minor) #62

Closed
wants to merge 1 commit into from

Conversation

depfu[bot]
Copy link
Contributor

@depfu depfu bot commented May 11, 2021


🚨 Your current dependencies have known security vulnerabilities 🚨

This dependency update fixes known security vulnerabilities. Please see the details below and assess their impact carefully. We recommend to merge and deploy this as soon as possible!


Here is everything you need to know about this update. Please take a good look at what changed and the test results before merging this pull request.

What changed?

✳️ codecov (3.6.5 → 3.8.2) · Repo · Changelog

Security Advisories 🚨

🚨 Command injection in codecov (npm package)

Impact

The upload method has a command injection vulnerability. Clients of the codecov-node library are unlikely to be aware of this, so they might unwittingly write code that contains a vulnerability.

A similar CVE was issued: CVE-2020-7597, but the fix was incomplete. It only blocked &, and command injection is still possible using backticks instead to bypass the sanitizer.

We have written a CodeQL query, which automatically detects this vulnerability. You can see the results of the query on the codecov-node project here.

Patches

3.7.1

Workarounds

None, however, the attack surface is low in this case. Particularly in the standard use of codecov, where the module is used directly in a build pipeline, not built against as a library in another application that may supply malicious input and perform command injection.

References

For more information

If you have any questions or comments about this advisory:

Release Notes

3.8.2

3.8.2

Fixes

  • #304 Add coverage-final.json as a possible coverage file during file lookup

3.8.1

v3.8.1

Fixes

  • #246 Revert "Bump teeny-request from 6.0.1 to 7.0.0"

3.8.0

v3.8.0

Features

  • #160 Add Github Actions support

Fixes

  • #173 Fix broken gcov command
  • #195 Update Node testing versions
  • #200 Remove flaky tests
  • #204 Create CHANGELOG and remove flaky v4 test
  • #208 Add license scan report and status
  • #220 Remove errant bitly

Dependencies

  • #189 Bump lint-staged from 10.0.7 to 10.2.11
  • #190 [Security] Bump handlebars from 4.5.3 to 4.7.6
  • #191 Bump prettier from 1.19.1 to 2.0.5
  • #192 Bump mock-fs from 4.10.4 to 4.12.0
  • #196 Bump teeny-request from 6.0.1 to 7.0.0
  • #197 Bump eslint-config-prettier from 4.3.0 to 6.11.0
  • #198 Bump js-yaml from 3.13.1 to 3.14.0
  • #199 Bump husky from 4.2.1 to 4.2.5
  • #202 Bump eslint from 5.16.0 to 7.7.0
  • #203 Bump jest from 24.9.0 to 26.4.1
  • #205 Bump mock-fs from 4.12.0 to 4.13.0
  • #206 Bump jest from 26.4.1 to 26.4.2
  • #207 Bump prettier from 2.0.5 to 2.1.0
  • #209 Bump lint-staged from 10.2.11 to 10.2.13
  • #210 Bump prettier from 2.1.0 to 2.1.1
  • #212 Bump eslint from 7.7.0 to 7.8.1
  • #214 Bump lint-staged from 10.2.13 to 10.3.0
  • #215 Bump husky from 4.2.5 to 4.3.0
  • #216 Bump node-fetch from 2.6.0 to 2.6.1
  • #217 Bump eslint from 7.8.1 to 7.9.0
  • #218 Bump prettier from 2.1.1 to 2.1.2
  • #219 Bump lint-staged from 10.3.0 to 10.4.0
  • #222 Bump eslint-config-prettier from 6.11.0 to 6.12.0
  • #223 Bump eslint from 7.9.0 to 7.10.0
  • #224 Bump teeny-request from 7.0.0 to 7.0.1

Does any of this look wrong? Please let us know.

Commits

See the full diff on Github. The new version differs by more commits than we can show here.


Depfu Status

Depfu will automatically keep this PR conflict-free, as long as you don't add any commits to this branch yourself. You can also trigger a rebase manually by commenting with @depfu rebase.

All Depfu comment commands
@​depfu rebase
Rebases against your default branch and redoes this update
@​depfu recreate
Recreates this PR, overwriting any edits that you've made to it
@​depfu merge
Merges this PR once your tests are passing and conflicts are resolved
@​depfu close
Closes this PR and deletes the branch
@​depfu reopen
Restores the branch and reopens this PR (if it's closed)
@​depfu pause
Ignores all future updates for this dependency and closes this PR
@​depfu pause [minor|major]
Ignores all future minor/major updates for this dependency and closes this PR
@​depfu resume
Future versions of this dependency will create PRs again (leaves this PR as is)

@codecov
Copy link

codecov bot commented May 11, 2021

Codecov Report

Merging #62 (6f05db2) into master (b28cb1a) will increase coverage by 92.63%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #62       +/-   ##
===========================================
+ Coverage        0   92.63%   +92.63%     
===========================================
  Files           0        4        +4     
  Lines           0       95       +95     
  Branches        0        9        +9     
===========================================
+ Hits            0       88       +88     
- Misses          0        7        +7     
Impacted Files Coverage Δ
index.js 84.78% <0.00%> (ø)
lib/generate-build-file.js 100.00% <0.00%> (ø)
lib/generate-asset-links-file.js 100.00% <0.00%> (ø)
lib/generate-icons.js 100.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b28cb1a...6f05db2. Read the comment docs.

@depfu
Copy link
Contributor Author

depfu bot commented Jul 27, 2021

Closed in favor of #66.

@depfu depfu bot closed this Jul 27, 2021
@depfu depfu bot deleted the depfu-update-npm-codecov-3.8.2 branch July 27, 2021 08:05
@depfu
Copy link
Contributor Author

depfu bot commented Jul 27, 2021

Closed in favor of #66.

2 similar comments
@depfu
Copy link
Contributor Author

depfu bot commented Jul 27, 2021

Closed in favor of #66.

@depfu
Copy link
Contributor Author

depfu bot commented Jul 27, 2021

Closed in favor of #66.

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

Successfully merging this pull request may close these issues.

0 participants