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

Application can render javascript on pages that make use of javascript and include XSS vulnerabilites #334

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

Conversation

Yusuf-YENICERI
Copy link

I have realized that application can't be able to scan websites that make use of javascript to render the requested query.

What does it implement/fix? Explain your changes.

I have added the ability to render the javascript included in page which results in ability to scan javascript powered pages.

Where has this been tested?

Python Version:3.7.3
Operating System: Debian 10

Does this close any currently open issues?

#290

Does this add any new dependency?

requests_html package

Does this add any new command line switch/option?

--javascript

Any other comments you would like to make?

I hope the pull request is going to be merged, inshaAllah.

Some Questions

  • [+ ] I have documented my code.
  • [ +] I have tested my build before submitting the pull request.

“Yusuf-YENICERI” and others added 2 commits October 18, 2021 17:19
@ItsIgnacioPortal
Copy link

Since @s0md3v seems to have abandoned this project, I've gone ahead and took the liberty to make a fork: XSStrike-Reborn
Your pull request has been merged into that project. Your efforts are appreciated 😁

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.

3 participants