Skip to content

doc: improve security section of README.md#17929

Closed
Trott wants to merge 1 commit intonodejs:masterfrom
Trott:fix-security-paragraph
Closed

doc: improve security section of README.md#17929
Trott wants to merge 1 commit intonodejs:masterfrom
Trott:fix-security-paragraph

Conversation

@Trott
Copy link
Copy Markdown
Member

@Trott Trott commented Jan 1, 2018

  • 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)
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

doc

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Issues and PRs related to the documentations. security Issues and PRs related to security.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants