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 ReDoS lab #561

Merged
merged 2 commits into from
Jul 24, 2024
Merged

Add ReDoS lab #561

merged 2 commits into from
Jul 24, 2024

Conversation

camilaavilarinho
Copy link
Contributor

Copy link
Contributor

@david-a-wheeler david-a-wheeler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall this is GREAT! I have a few nits, see my specific comments.

docs/labs/redos.html Outdated Show resolved Hide resolved
docs/labs/redos.html Outdated Show resolved Hide resolved
docs/labs/redos.html Outdated Show resolved Hide resolved
docs/labs/redos.html Outdated Show resolved Hide resolved
docs/labs/redos.html Outdated Show resolved Hide resolved
docs/labs/redos.html Outdated Show resolved Hide resolved
docs/labs/redos.html Outdated Show resolved Hide resolved
@david-a-wheeler
Copy link
Contributor

Thank you so much! I have lots of little nits, but they're nits. This looks great!

@camilaavilarinho
Copy link
Contributor Author

Thank you so much! I have lots of little nits, but they're nits. This looks great!

Thank you for your review! I missed the github notification, but I'm working on the changes.

Signed-off-by: Camila Vilarinho <camilaavilarinho@gmail.com>
Signed-off-by: Camila Vilarinho <camilaavilarinho@gmail.com>
@camilaavilarinho camilaavilarinho changed the title [Draft] Add ReDoS lab Add ReDoS lab Jul 18, 2024
@david-a-wheeler david-a-wheeler merged commit f39c036 into ossf:main Jul 24, 2024
2 checks passed
@camilaavilarinho camilaavilarinho deleted the add-redos-lab branch July 26, 2024 12:30
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

3 participants