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

Remove workaround no longer needed #298

Merged
merged 1 commit into from
Feb 10, 2025
Merged

Conversation

thc202
Copy link
Member

@thc202 thc202 commented Feb 10, 2025

Remove workaround that's now causing exceptions and it's no longer needed, support for PhantomJS was removed a while ago (#240).

Remove workaround that's now causing exceptions and it's no longer
needed, support for PhantomJS was removed a while ago (zaproxy#240).

Signed-off-by: thc202 <[email protected]>
@thc202 thc202 added this to the 0.24.0 milestone Feb 10, 2025
@psiinon
Copy link
Member

psiinon commented Feb 10, 2025

Logo
Checkmarx One – Scan Summary & Details79680c56-cc88-4cbb-8732-d805c65f7422

New Issues (1)

Checkmarx found the following issues in this Pull Request

Severity Issue Source File / Package Checkmarx Insight
HIGH CVE-2024-57699 Maven-net.minidev:json-smart-2.5.1
detailsDescription: A security issue was found in Netplex Json-smart 2.5.0 through 2.5.1. When loading a specially crafted JSON input, containing a large number of '{'...
Attack Vector: NETWORK
Attack Complexity: LOW
Vulnerable Package

@kingthorin kingthorin merged commit 8cb45f6 into zaproxy:main Feb 10, 2025
6 checks passed
@thc202 thc202 deleted the rm-quit branch February 10, 2025 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants