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

Update to Log4j2 and usage of dependenciesManagement #8816

Closed
2 of 6 tasks
taba90 opened this issue Nov 15, 2022 · 4 comments · Fixed by #8815
Closed
2 of 6 tasks

Update to Log4j2 and usage of dependenciesManagement #8816

taba90 opened this issue Nov 15, 2022 · 4 comments · Fixed by #8815
Assignees
Milestone

Comments

@taba90
Copy link
Contributor

taba90 commented Nov 15, 2022

Description

This issue is to track on mapstore board the upgrade to log4j2 and the generalization of usage for dependenciesManagement in projects pom.xml files.

What kind of improvement you want to add? (check one with "x", remove the others)

  • Minor changes to existing features
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

Other useful information

@taba90
Copy link
Contributor Author

taba90 commented Nov 15, 2022

Opened pr on master branch for:

@taba90
Copy link
Contributor Author

taba90 commented Nov 15, 2022

pr on geoorchestra will be opened if this modifications will ever land on stable branch since georchestra is not using master.

@taba90 taba90 self-assigned this Nov 15, 2022
@tdipisa tdipisa added this to the 2023.01.00 milestone Dec 16, 2022
@offtherailz offtherailz mentioned this issue Feb 3, 2023
15 tasks
@offtherailz
Copy link
Member

This issue has been merged #8815. Logs works as expected. So I am going to close it.

@offtherailz
Copy link
Member

Accepted because logs works as expected

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants