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

Addition of a Checklist - Create The Essential Security Checklist for… #984 #1460

Closed
wants to merge 2 commits into from

Conversation

SNCherry
Copy link

@SNCherry SNCherry commented Aug 2, 2024

No description provided.

@jmanico
Copy link
Member

jmanico commented Aug 3, 2024

I am ok with this general checklist. But can we add do this is a way that references all of the other cheatsheets? The main list of cheatsheets is here. https://cheatsheetseries.owasp.org/ I would prefer that each sections links to the relevant cheatsheet or cheatsheets...

@kwwall
Copy link
Collaborator

kwwall commented Aug 3, 2024

I was just about to say, this looks like an almost verbatim rehash of what's mentioned on the referenced link, https://www.yslingshot.com/security-checklist/. As such, this looks too much like trying to take advantage of OWASP Cheat Sheats as your personal ad agency. I'm not sure the OWASP board would approve, especially since Slingshot is a commercial endeavor selling services (see https://www.yslingshot.com/services/) and not a non-profit. So, as this is presented, I think it needs extensive rewrite. (I also am generally not in favor of checklists. We have plenty of those already. If one is going to take a checklist approach, then the OWASP ASVS would be a better reference for this, very vague checklist.)

Just my $.02. But at a minimum, I think this needs to do what @jmanico is suggesting and reference the other Cheat Sheets in the relevant sections rather than pointing to https://www.yslingshot.com/security-checklist/.

@jmanico
Copy link
Member

jmanico commented Aug 3, 2024

Thank you Kevin. Unless this is refactored in a very significant way, I am going with Kevin's perspective on this.

@szh
Copy link
Collaborator

szh commented Aug 8, 2024

Given the above comments, I'll close this for now. Feel free to reopen if major changes are made that would change the decision.

@szh szh closed this Aug 8, 2024
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.

4 participants