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

Use a headless javascript browser. Stop using ClientApiMain as its o… #14

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

Conversation

ilatypov
Copy link

@ilatypov ilatypov commented Jan 9, 2017

…ld code chokes on absence of "reliability" tags in alerts that turned to "confidence" ones. The new ClientApiMain has no "spider" command.

…d code chokes on absence of "reliability" tags in alerts that turned to "confidence" ones. The new ClientApiMain has no "spider" command.
@ilatypov
Copy link
Author

The following link shows the above change factoring out white space changes,

594b44f?w=1

…s from a fresh environment. Avoid forking new JVMs. Deploy the application using Tomcat Ant tasks because deleting the unpacked WAR fails when Tomcat has a JAR open inside the web app. Swallow the exception text of the alert check as it repeats the list of alerts already shown in Ant stodout.
@ilatypov
Copy link
Author

The above commit also terminates Selenium's phantomjs.exe and ZAP's java.exe gracefully.

4f66722?w=1

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.

1 participant