search

fullstorydev / solr-bench

Solr benchmarking and load testing harness
Apache License 2.0
17 stars 10 forks source link

[Snyk] Security upgrade org.json:json from 20180130 to 20230227 #69

Closed snyk-bot closed 1 year ago

snyk-bot commented 1 year ago

Snyk has created this PR to fix one or more vulnerable packages in the `maven` dependencies of this project.

Changes included in this PR

Vulnerabilities that will be fixed

With an upgrade:
Severity Priority Score (*) Issue Upgrade Breaking Change Exploit Maturity
high severity 696/1000
Why? Proof of Concept exploit, Has a fix available, CVSS 7.5		 Denial of Service (DoS)
SNYK-JAVA-ORGJSON-5488379		 org.json:json:
20180130 -> 20230227
Yes Proof of Concept

(*) Note that the real score may have changed since the PR was raised.

Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.

For more information: 🧐 View latest project report

🛠 Adjust project settings

📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:

🦉 Denial of Service (DoS)

patsonluk commented 1 year ago

Tried to test locally, but only 2 pieces of logics use this library:

  1. After restart ping "http://"+restartNode.getNodeName() + "/api/node/heap" to get heap stats
  2. MetricsCollector on zk and solr metrics

Test results:

  1. the endpoint does not exist for our solr instance, so cannot test it.
  2. Used "metrics": [ "jvm/solr.jvm/memory.heap.usage", "jvm/solr.jvm/threads.count" ] and it parsed fine

So the upgrade should be good! Going to merge it