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 test section for Clickjacking #1139

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

Conversation

doverh
Copy link
Contributor

@doverh doverh commented May 3, 2024

This PR covers issue #183

  • This PR handles the issue and requires no additional PRs.
  • You have validated the need for this change.

What did this PR accomplish?

  • Removes duplicated content available on OWASP Clickjacking Defense Cheat Sheet.

  • Removes POC and start to identify tests that can be implemented for Clickjacking.

  • Review How to Test section

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

@kingthorin
Copy link
Collaborator

Well start with the bot feedback 😉

This comment was marked as resolved.

@kingthorin
Copy link
Collaborator

I'll try to lookup some info in that Safari item, just don't know when I'll have a good chance.

@doverh
Copy link
Contributor Author

doverh commented May 8, 2024

Thanks @kingthorin! My initial intention was to allow more contributors to review and add to this review. Should we post on wstg slack channel asking for contributions with the content?

@kingthorin
Copy link
Collaborator

Sure, can't hurt to ask for more eyes.

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.

None yet

2 participants