-
Notifications
You must be signed in to change notification settings - Fork 4.3k
Add Cheat Sheet: NoSQL Security Cheat Sheet #1852
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
|
there was a mistake on PR name, fixed it |
|
A ot of the content overlaps with https://github.com/OWASP/CheatSheetSeries/blob/master/cheatsheets/Database_Security_Cheat_Sheet.md I think it is better to update that cheatsheet than try to keep both of them up to date |
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 thought it is a better idea to cover only nosql databases on separated cheat sheet. Because it'll make it too long maybe complex for the developer. We can also update DB Security cheat sheet to cover only relational dbs (I can handle it) and give ref to NoSQL one. wdyt? @mackowski |
|
I think a separate cheetsheet is a good idea. Can you also remove the NoSQL info from the database cheatsheet and link to your new cheatsheets? I think smaller, focused cheatsheets are a good idea. Is this ok with you @mackowski ? |
I’ve created an issue for this and will take care of it. I’ll also review the Database Security Cheat Sheet to ensure it contains all necessary things and up to date. is it ok for you? @jmanico |
|
I like this plan. |
|
I like this plan! |
|
can we merge this one? so that I can give link to the other cheat sheet :) @mackowski |
|
Merged! |
I've added new cheat sheet about NoSQL security.
It contains most popular NoSQL vulnerabilities and preventations.
If your PR is related to an issue, please finish your PR text with the following line:
This PR fixes issue #1851.