Issue #25 Added ability to get the report filename

[BretMcDanel]

When automating it may be advantageous to know with high certainty what the report filename is for further processing (eg XML -> anything).

This changes the local variable report_file_name to a class property report_file_name.

eg:
burp = Burpa( ... )
burp.scan( ... )
do_more_processing_on_report_file(burp.report_file_name)

[tristanlatr]

Hello @BretMcDanel,

Thanks for your PR, but I think the implementation is not taking care of all edge cases. A scan can be run on several URLs, generating several report files. I think the way to handle this usage of the scan() method is to return a Report object that will contain status, metrics, issues and report filename for every scans.

Are you willing to implement such a change?

[BretMcDanel]

Yeah, I can look at that.

Off the cuff, an array of objects that contain at least:

• Url
• report path

Do you want anything else returned? It feels superfluous to include the report contents given they are saved to a file. Minimal parsing of the response could be done, for example counting issues (which would give a positive response that no issues were found, as opposed to no report being generated). Any other ideas?

[tristanlatr]

Thanks, this is good but just a couple of changes would be nice

[BretMcDanel]

I believe this will resolve everything. Thanks.

[tristanlatr]

I will apply some changes to this PR and merge it when I have a moment. Probably next week.