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.

Sign up for GitHub

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

Fix for #50 #52

Merged
merged 1 commit into from
Jul 23, 2018
Merged

Fix for #50 #52

merged 1 commit into from
Jul 23, 2018

Commits on Jul 17, 2018

  1. Fix for #50 

    This should resolve the issue described by: #50

Cause: Use of AND condition instead of OR condition means that user must be using a free Burp version AND have also not specified a project file. However, this ignores two cases:

1. The user mistakenly states a project file with Burp free edition
2. The user does not state a project file with Burp paid edition

Both of these cases require the creation of an empty projectData array.

Resolution: Changed conditional operator from `&&` to `||`.

I also rearranged the operands so that the `Note:` line flows into the logic of the if-statement. 
"Burp Free does not support project..." => "`if (!burpEdition.equalsIgnoreCase('free')`"
    @das-sein
    das-sein committed Jul 17, 2018
    Configuration menu
    Copy the full SHA
    c66088d View commit details
    Browse the repository at this point in the history