Skip to content

Commit 9197217

Browse files
TrottMylesBorins
authored andcommitted
doc: improve security section of README.md
* Remove fluff text and get to the point: Report security flaws to [email protected]. Please do not disclose security flaws publicly until they have been handled by the security team. * Fix somewhat confusing paragraph that says there are no "hard and fast rules" but then uses _must_ in the context of a "general rule". Easiest solution seems to be to change _must_ to _should_. * Minor style change (_you will_ instead of _you'll_) PR-URL: #17929 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Tiancheng "Timothy" Gu <[email protected]> Reviewed-By: Jon Moss <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
1 parent a17c3e6 commit 9197217

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

README.md

+8-9
Original file line numberDiff line numberDiff line change
@@ -169,20 +169,19 @@ officially supported platforms.
169169

170170
## Security
171171

172-
All security bugs in Node.js are taken seriously and should be reported by
173-
emailing [email protected]. This will be delivered to a subset of the project
174-
team who handle security issues. Please don't disclose security bugs
175-
publicly until they have been handled by the security team.
172+
Security flaws in Node.js should be reported by emailing [email protected].
173+
Please do not disclose security bugs publicly until they have been handled by
174+
the security team.
176175

177-
Your email will be acknowledged within 24 hours, and you’ll receive a more
176+
Your email will be acknowledged within 24 hours, and you will receive a more
178177
detailed response to your email within 48 hours indicating the next steps in
179178
handling your report.
180179

181180
There are no hard and fast rules to determine if a bug is worth reporting as
182-
a security issue. The general rule is any issue worth reporting
183-
must allow an attacker to compromise the confidentiality, integrity
184-
or availability of the Node.js application or its system for which the attacker
185-
does not already have the capability.
181+
a security issue. The general rule is an issue worth reporting should allow an
182+
attacker to compromise the confidentiality, integrity, or availability of the
183+
Node.js application or its system for which the attacker does not already have
184+
the capability.
186185

187186
To illustrate the point, here are some examples of past issues and what the
188187
Security Response Team thinks of them. When in doubt, however, please do send

0 commit comments

Comments
 (0)