Skip to content

Conversation

@cjdcordeiro
Copy link
Collaborator

  • Have you signed the CLA?

Add a security policy (SECURITY.md file) to the repo.

@cjdcordeiro cjdcordeiro added the Simple Nice for a quick look on a minute or two label Jan 7, 2025
@zhijie-yang
Copy link
Contributor

This is a simple security.md yet provides enough information for users to submit vulnerability findings. I'm just wondering if we should adopt a similar structure as the one in the multipass repo.

Even though we want to keep it simple, I would suggest adding the title "Security Policy" and the header "Reporting a Vulnerability" before the proposed contents.

@cjdcordeiro
Copy link
Collaborator Author

This is a simple security.md yet provides enough information for users to submit vulnerability findings. I'm just wondering if we should adopt a similar structure as the one in the multipass repo.

Even though we want to keep it simple, I would suggest adding the title "Security Policy" and the header "Reporting a Vulnerability" before the proposed contents.

This is actually the template that is being mandated by Security and already adopted by many repos.

As for the Title and subtitle, they feel a bit redundant in the absence of other sections but I don't mind adding them (done).

Copy link
Contributor

@zhijie-yang zhijie-yang left a comment

Choose a reason for hiding this comment

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

Thanks! This looks nice to me.

Copy link
Collaborator

@letFunny letFunny left a comment

Choose a reason for hiding this comment

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

Thanks for this!

@cjdcordeiro cjdcordeiro merged commit 03f5597 into canonical:main Jan 9, 2025
14 of 15 checks passed
@cjdcordeiro cjdcordeiro deleted the ROCKS-1500/security-policy branch January 9, 2025 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Simple Nice for a quick look on a minute or two

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants