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

security-guidelines: add advice on keeping vulns off public issue tracker #423

Merged
merged 3 commits into from
Jan 6, 2021

Conversation

mikesamuel
Copy link
Contributor

This reboots PR #315.
It differs so as to address @wesleytodd's concerns:

  • Inlines URLs for consistency with the surrounding markdown.
  • Replaces discussion of reputation systems with discussion of
    defining bounds and adds an example of such a definition.
  • Moves examples into existing table of example policy documents.

…cker

This reboots PR nodejs#315 and tries to address @wesleytodd's changes.
- Inlines URLs
- Replaces discussion of reputation systems with a link to an
  example of defining bounds for security researchers.
- Moves example into existing table.
Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Co-authored-by: Michael Dawson <mdawson@devrus.com>
@dominykas
Copy link
Member

3 weeks since 4th approval - merging.

@dominykas dominykas merged commit 7ceb39e into nodejs:main Jan 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants