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

dependency update #669

Closed
wants to merge 8 commits into from
Closed

dependency update #669

wants to merge 8 commits into from

Conversation

dobrawek
Copy link

I've started working with photon - I've updated all dependencies to the lastest and changed usage of com.vividsolutions.jts to org.locationtech.jts that is now a part of latest postgis

@lonvia
Copy link
Collaborator

lonvia commented Apr 19, 2022

Unlikely to work because ES 5.6 relies on com.vividsolutions.jts

@avshabavsha
Copy link

Is it possible to exclude the versions update that are uncertain of and approve the rest?

Specifically, I planned to create a PR to update log4j to 2.17.2 (same version as @dobrawek did), but noticed it's already part of this PR.

This is to resolve these two issues:
https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2021-44832
https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2021-45105

@lonvia
Copy link
Collaborator

lonvia commented Apr 20, 2022

See #620 for the mentioned CVEs. They have been long since resolved.

@lonvia lonvia closed this Jul 19, 2022
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.

3 participants