-
Notifications
You must be signed in to change notification settings - Fork 5
Created Security.md file for TheCodeVerseHub for vulns in code! #4
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
Conversation
…ty reporting I PhillsPhanbh3 has made a security policy for reporting ANY security vulnerabilities to the development team, so that patches can be made within a week of acknowledgment!
sinisterMage
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
first of all, thanks for willing to contribute to the cvh-Linux, but the tone of the security.md you suggested is a bit unprofessional, especially line 9, could you modify it slightly to be more professional, and again, thank you 😄
SECURITY.md
Outdated
|
|
||
| ## Reporting a Vulnerability | ||
|
|
||
| You have discovered a SECURITY VULNERABILITY! OH NO! WHAT'S NEXT?! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i personally am not a fan of this tone, could you change it into a slightly more professional looking one?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yea same i agree with Nova
youngcoder45
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can be modified but yea thanks
|
I have received your requests and will get them done tonight, I just wanted some ideas on it because it is my 1st time, so more professional, got it (: |
Updated the language for reporting security vulnerabilities and added acknowledgment and response process details.
|
I have just updated the security.md page @sinisterMage @youngcoder45 to make it look more professional (even though some days my spelling isn't), and if you want to get further edits, you can gladly let me know! |
|
LGTM, but @youngcoder45 its up to you |
|
forgot to re request the review i'm sorry @sinisterMage I just wasn't sure if it would cause a merge issue then |
sinisterMage
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks much better now, @youngcoder45 up to you
youngcoder45
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yea it's good enough now it's good to merge
|
Thank you @youngcoder45 and @sinisterMage for allowing me to help yous out and even helping me out! |
I PhillsPhanbh3 has made a security policy for reporting ANY security vulnerabilities to the development team, so that patches can be made within a week of acknowledgment!
I am VERY open to edit requests and way more! Please give me ideas on how to give suggestions for security.md files as this is my 1st time doing so!