Skip to content
This repository has been archived by the owner on Apr 11, 2021. It is now read-only.

requirements.txt and updated README #5

Open
pwnetrationguru opened this issue Feb 13, 2015 · 1 comment
Open

requirements.txt and updated README #5

pwnetrationguru opened this issue Feb 13, 2015 · 1 comment

Comments

@pwnetrationguru
Copy link

Hey,

Hope you're well. Was going to play around with this, but it's not clear what packages the python scripts need nor how to setup domxssscanner.

It would be good to include requirements.txt file so people can use a python virtualenv and install the packages need by domxssscanner with pip install -r requirements.txt. A simple way to generate that would be to do a pip freeze where you do have domxssscanner working. Be warned, that might include packages that have been installed on the system that aren't necessary for domxssscanner.

Also, it would be great if the README included documentation on how best to setup. A process like the following would be great:

  • clone domxssscanner repo
  • virtualenv env
  • . ./env/bin/activate
  • pip install -r requirements.txt
  • find . -name "*.js" | xargs ./domxss.py
    • that is an obviously made up command since I am unfamiliar with how to run domxssscanner
@yaph
Copy link
Owner

yaph commented Feb 15, 2015

You're right without this info the project setup is hard. At the moment I have no local setup myself, but the live project is hosted on Google App Engine. I actually want to port it to flask and move away from App Engine, when I find the time to do so, I'll create a requirements file.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants