Skip to content

Commit

Permalink
feat: mention npm audit in dependency security reports post (#245)
Browse files Browse the repository at this point in the history
* feat: mention npm audit in dependency security reports post

* lint:spelling
  • Loading branch information
JoshuaKGoldberg authored Mar 4, 2024
1 parent d4dbf0b commit 9f0c274
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
3 changes: 2 additions & 1 deletion cspell.json
Original file line number Diff line number Diff line change
Expand Up @@ -101,6 +101,7 @@
"dotfile",
"dprint",
"dtslint",
"Durrant",
"dustinspecker",
"Dweck",
"Eghbal",
Expand All @@ -126,8 +127,8 @@
"FOSS",
"fourslash",
"Frankensteinian",
"Freakin",
"Freakin'",
"Freakin",
"freezeframe",
"Freitas",
"fullscreenmario",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,8 @@ Unless you can demonstrate a real security vulnerability in one of my packages,
These excessive reports are produced because traditional scanners such as `npm audit` and Dependabot don't have a way to indicate which API(s) in a package are impacted by a reported issue.
That means every downstream package that directly or transitively relies on an impacted package -even if they never use the impacted APIs- will get a security report.

> 👉 See [npm audit: Broken by Design](https://overreacted.io/npm-audit-broken-by-design) for a deep dive into `npm audit`'s flaws.
Much of the time, the vulnerability is only exploitable if a user runs some carefully crafted malicious code on the server the package is running on.
That can be very important to patch for some users of some packages!

Expand Down Expand Up @@ -103,3 +105,4 @@ If you can, I'd recommend instead spending the time to learn how the code in pla
## Acknowledgements

Thanks to [Lydia Stepanek](https://www.lydiaoncybersecurity.com) for reviewing the post and providing helpful feedback & suggestions! 🤝
Thanks also to [Ben Durrant](https://github.com/EskiMojo14) for [mentioning the `npm audit` deep dive](https://www.reddit.com/r/javascript/comments/1b6fqvt/comment/ktbpoa7).

0 comments on commit 9f0c274

Please sign in to comment.