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

Upgrade of slf4j-log4j12 dependency #422

Merged
merged 2 commits into from
Nov 1, 2023

Conversation

chenejac
Copy link
Contributor

VIVO GitHub issue: 3913

What does this pull request do?

Resolve security vulnerabilities with log4j 1.2.17

What's new?

Upgrade slf4j-log4j12 version in pom.xml file

How should this be tested?

  • mvn clean install
  • check whether tests passing
  • run vivo, turn on developer panel and logging in, check whether there is any issue with logging

Interested parties

Tag (@ mention) interested parties or, if unsure, @VIVO-project/vivo-committers

Copy link
Contributor

@litvinovg litvinovg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Works fine.

@chenejac chenejac merged commit b21c026 into vivo-project:main Nov 1, 2023
1 check passed
@chenejac chenejac linked an issue Nov 13, 2023 that may be closed by this pull request
chenejac added a commit that referenced this pull request Mar 18, 2024
* Upgrade of slf4j-log4j12 dependency (to 1.7.36)

* Optimization of imports
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.

Log4j security vulnerability
3 participants