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

Update dom-xss-web-message.yaml #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

pookie-magarmach
Copy link

there is an error in which is being fixed by me

@JoshMorrison99
Copy link
Owner

Thank you for reviewing, but this is not an error. The template is searching for web messages which are usually in the form:

window.addEventListener('message', function(e) {

By searching for just window.addEventListener('message', I'm able to check if a web message is on the page.

If you are searching for postMessage based bugs, I highly recommend this extension. It will help out a lot. https://github.com/fransr/postMessage-tracker

I have an example of how I used it on a Portswigger lab writeup here: https://shelled.io/writeups/portswigger/dom-xss-using-web-messages.html

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