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

Better Fix #50 and #46 #63

Merged
merged 5 commits into from
Oct 24, 2018
Merged

Better Fix #50 and #46 #63

merged 5 commits into from
Oct 24, 2018

Conversation

adracea
Copy link
Contributor

@adracea adracea commented Aug 31, 2018

Adds :

  • -PappArgs="['arg1','arg2','arg3']" to be able to pass arguments to burp suite directly via the gradle wrapper ( gradlew bootRun -PappArgs="['--project-file=./test.burp','--config-file=./cfg.json','--user-config-file=./usr-cfg.json']" ).
  • Fixed Can't start Burp-rest-api with own project-file #46 by changing what Fix for #50 #52 introduced by changing the negative assertions to positive assertions and and making the logical operator back to &&

Adding a bit of info on -PappArgs option
Changing the project file loading to be done only if the burp suite is pro edition AND the project file is mentioned , using positive assertions instead of negatives .
@vmwclabot
Copy link
Member

@adracea, you must sign our contributor license agreement before your changes are merged. Click here to sign the agreement. If you are a VMware employee, read this for further instruction.

@vmwclabot
Copy link
Member

@adracea, VMware has approved your signed contributor license agreement.

@adracea
Copy link
Contributor Author

adracea commented Sep 3, 2018

I've tested these and made the changes according to my "field" data . Burp free edition was started successfully in an Arch VM box and Burp Pro works quite fine with the new added options in 975afa8 for project file loading .

@ikkisoft
Copy link
Collaborator

I will review soon and merge. Btw, we're removing Burp free support thus you may want to update your PR as soon as the change is in master.

@ikkisoft
Copy link
Collaborator

Burp Free/Community is now deprecated. See #71

If you can adjust this pull request, I will merge your change to master.

Thanks again for the contribution!

@adracea
Copy link
Contributor Author

adracea commented Oct 23, 2018 via email

@adracea
Copy link
Contributor Author

adracea commented Oct 23, 2018

Okay , should be good now :D I reverted the commits to burpservice.java since those were the only blockers :)

@ikkisoft ikkisoft merged commit e96117d into vmware:master Oct 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants