Navigation Menu

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

Jump to bottom

Added --output-dir option to allow specifying directory for storing files #53

Closed
wants to merge 1 commit into from
Closed

Added --output-dir option to allow specifying directory for storing files #53

wants to merge 1 commit into from

Conversation

lllama
Copy link

@lllama lllama commented Jul 2, 2012

  • _sqlmap.py
    • move command line parsing to earlier in main to allow setPaths to know output_dir.
    • Change cmdLineOptions update to reflect earlier parsing
  • lib/core/common.py
    • setPaths is now passed the command line options so they can be used for setting the path.
  • lib/core/cmdline.py
    • added OptionParser option for the new option.

@lllama
Added --output-dir option to allow specifying directory for storing f…
0b5032c 
…iles.

* _sqlmap.py - move command line parsing to earlier in main function
               to allow setPaths to know output_dir.
               Change cmdLineOptions update to reflect earlier parsing
* lib/core/common.py - setPaths is now passed the command line options
                       so they can be used for setting the path.
* lib/core/cmdline.py - added OptionParser option for the new option.
@lllama
Copy link
Author

lllama commented Jul 2, 2012

Changing the setting of paths.SQLMAP_OUTPUT_PATH from:
paths.SQLMAP_OUTPUT_PATH = options.output_dir
to
paths.SQLMAP_OUTPUT_PATH = os.path.abspath(os.path.expanduser(options.output_dir))

Would probably be useful. Will attempt another pull request once github start accepting my pushes again.

stamparm added a commit that referenced this pull request Jul 2, 2012
@stamparm
Adding switch --output-dir (Issue #53)
168aead
@stamparm stamparm closed this Jul 2, 2012
@ghost ghost assigned stamparm Jul 4, 2012
@chym chym mentioned this pull request Nov 10, 2012
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@stamparm stamparm

Labels
None yet
Projects
None yet
Milestone
1.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@lllama @stamparm