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 XSS owasp 2017 rake-my-cake rails app #602

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Perkles
Copy link

@Perkles Perkles commented Oct 21, 2022

This PR add a XSS owasp 2017 Rails app following #438.

Since this PR was open way back in 2020 referring Owasp 2017 I end up putting the code inside a folder called owasp-top10-2017-apps, let me know if i did it right.

I also took the liberty of using part of the exemple of Gossip World specifically the GO Keylogger for lack of imagination.

@fguisso
Copy link
Member

fguisso commented Oct 25, 2022

Hey @Perkles thanks for the contribution, I'll review it soon. Can you change the directory names and tags inside the challenge to follow the new OWASP top 10 2021?

Look at this image, XSS A7-2017 is now inside A3-2022.
image

@Perkles
Copy link
Author

Perkles commented Oct 25, 2022

Sure thing @fguisso.
All changed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants