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

Add MASVS index #853

Merged
merged 2 commits into from
Feb 14, 2022
Merged

Add MASVS index #853

merged 2 commits into from
Feb 14, 2022

Conversation

cpholguera
Copy link
Contributor

Added MASVS index mapping to useful Cheat Sheets.

@mackowski
Copy link
Collaborator

Wow, nice :) thank you @cpholguera - this PR also show that we need to add some content for V6 and V8

@mackowski
Copy link
Collaborator

To add new page (outside of cheatsheets directory) we also need to update generating the MkDocs. I think that these three PR can help with that:

@cpholguera
Copy link
Contributor Author

Hi @mackowski, I've fixed all generation scripts, please let me know if something else is needed.

Regarding V6 and V8, absolutely! and we will address that, maybe we need some new ones or we can also contribute to others (as in #852) so that we don't have duplicated content. In the MSTG/MASVS we want to rely more on other standards and in the Cheat Sheets.

@mackowski
Copy link
Collaborator

Looks good. I will merge and build it.

@mackowski mackowski merged commit 26477c2 into OWASP:master Feb 14, 2022
@mackowski
Copy link
Collaborator

@cpholguera it is live https://cheatsheetseries.owasp.org/IndexMASVS.html :-)

@cpholguera
Copy link
Contributor Author

Awesome @mackowski, thanks a lot! We'll announce it soon in our official accounts 🙌

@cpholguera
Copy link
Contributor Author

After seeing it rendered I noticed some minor things to be fixed. Here they are fixed: #856

@mackowski
Copy link
Collaborator

@cpholguera #856 was merged and should be in prod

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.

2 participants