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

Update Readme, Correct Missing Driver File Issue, Correct Driver File Settings and Interface #1

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

phefley
Copy link

@phefley phefley commented Sep 12, 2019

Hello!! The readme incorrectly mentioned CORS and should have mentioned CSP. Also, users were encountering issues when the chromedriver file was not present due to null pointer issues. Exception handling was corrected. Thank you!

Peter Hefley and others added 4 commits September 12, 2019 09:25
	modified:   build.gradle
	renamed:    javascript-security-extension-all-1.1.jar -> javascript-security-extension-all-1.1.1.jar
	modified:   src/main/java/org.focalpoint.isns.burp.srichecks/DriverServiceManager.java
Corrected issues when the chromedriver path was not available at start

	modified:   javascript-security-extension-all-1.1.1.jar
	modified:   src/main/java/org.focalpoint.isns.burp.srichecks/DriverServiceManager.java
	renamed:    javascript-security-extension-all-1.1.1.jar -> javascript-security-extension-all-1.1.2.jar
	modified:   src/main/java/burp/BurpExtender.java
	modified:   src/main/java/org.focalpoint.isns.burp.srichecks/PluginConfigurationTab.java
settings.
	modified:   javascript-security-extension-all-1.1.2.jar
	modified:   src/main/java/burp/BurpExtender.java
	modified:   src/main/java/org.focalpoint.isns.burp.srichecks/DriverServiceManager.java
@phefley
Copy link
Author

phefley commented Oct 10, 2019

Added changes to address issue phefley#2 for the chromedriver setting falling back to default at each load. This is now stored as a setting in burp preferences.

phefley and others added 6 commits October 28, 2019 15:58
cross-domain (DOM) issue.

Updated tests

	modified:   .gitignore
	modified:   build.gradle
	deleted:    javascript-security-extension-all-1.1.2.jar
	modified:   src/main/java/burp/BurpExtender.java
	modified:   src/main/java/org.focalpoint.isns.burp.srichecks/ScriptFinder.java
	modified:   src/test/java/org/focalpoint/isns/burp/srichecks/ScriptFinderTest.java
Try to obtain any headers available from the request and set
cookies, where present.

modified:   build.gradle
modified:   src/main/java/burp/BurpExtender.java
modified:   src/main/java/org.focalpoint.isns.burp.srichecks/ScriptFinder.java
Addressed issues with chromedriver binaries not loading and causing the
extension to be unusable.

	modified:   build.gradle
	modified:   src/main/java/org.focalpoint.isns.burp.srichecks/DriverServiceManager.java
	modified:   src/main/java/org.focalpoint.isns.burp.srichecks/PluginConfigurationTab.java
field

	modified:   src/main/java/org.focalpoint.isns.burp.srichecks/PluginConfigurationTab.java
@phefley
Copy link
Author

phefley commented Mar 23, 2022

Fixed issues where it was not possible to set the driver path location.

@phefley phefley changed the title Update Readme and Correct Missing Driver File Issue Update Readme, Correct Missing Driver File Issue, Correct Driver File Settings and Interface Mar 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant