DanMcInerney / xsscrapy

XSS spider - 66/66 wavsep XSS detected
1.66k stars 442 forks source link

Output to domain specific file #22

Closed hslatman closed 9 years ago

hslatman commented 9 years ago

The default implementation outputs to xsscrapy-vulns.txt, possibly mixing results for various domains which can make analysis of results harder. This PR changes the default behavior to write to domain specific files instead.

DanMcInerney commented 9 years ago

Thanks for the PR. I will verify it's all working and merge sometime this week.

On Sat, Aug 1, 2015 at 2:22 PM, hslatman notifications@github.com wrote:

The default implementation outputs to xsscrapy-vulns.txt, possibly mixing results for various domains which can make analysis of results harder. This

PR changes the default behavior to write to domain specific files instead.

You can view, comment on, or merge this pull request online at:

https://github.com/DanMcInerney/xsscrapy/pull/22 Commit Summary

  • Make output to file depending on domain that's being tested
  • Fix bug where item does not have a resp key

File Changes

Patch Links:

— Reply to this email directly or view it on GitHub https://github.com/DanMcInerney/xsscrapy/pull/22.

hslatman commented 9 years ago

A cleaner version of what I was trying to do on https://github.com/DanMcInerney/xsscrapy/pull/24