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

--path-report does not seem to work as documented #20

Closed
milosdjakonovic opened this issue Jun 5, 2021 · 1 comment
Closed

--path-report does not seem to work as documented #20

milosdjakonovic opened this issue Jun 5, 2021 · 1 comment
Labels
enhancement New feature or request

Comments

@milosdjakonovic
Copy link
Contributor

I'm sorry if I have missed something big but this option --path-report='' does not seem to work like declared:

--path-report <path>                           - Set report log file
                                                 [default: ./scanner-report.html]

If I run my scan like in example:

/usr/bin/php7.2 /opt/awscan.phar '/path/to/site' --report --path-report='/tmp/something' --report-format='txt' --disable-checksum --silent

I get:

  • /tmp/something isn't generated
  • /tmp/something.log is generated and is ok - has all the info needed

This two are not the same thing and may be confusing for user - for example, I don't know if it is a bug or poorly worded feature.

Suggestion: if this isn't a bug, improve wording: "Set report log file path and name. Note that name will be appended with .log extension.".

@marcocesarato marcocesarato added the enhancement New feature or request label Jun 6, 2021
@marcocesarato
Copy link
Owner

Hi, it's not a bug but with "Set Report Log File Path" means that you need also to set the filename, so I've added a better explanation of the option.
Thank you for your suggestion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants