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

Added lost slf4j, logback, log4j versions #8968

Closed
wants to merge 1 commit into from

Conversation

altro3
Copy link
Contributor

@altro3 altro3 commented Mar 18, 2023

Lost changes from micronaut 3.8 (see #8138)

@sdelamo
Copy link
Collaborator

sdelamo commented Mar 20, 2023

I think they should be in https://github.com/micronaut-projects/micronaut-platform not in core

@sdelamo sdelamo requested a review from melix March 20, 2023 08:19
@melix
Copy link
Contributor

melix commented Mar 20, 2023

They should probably neither be in core, nor in platform. The problem if you put them in core is that we're introducing unused dependencies in core. Obviously in platform then we would introduce a cyclic dependency between platform and core, so this is a no-go. There should probably be a micronaut-logging project for this.

@@ -207,7 +219,7 @@ kotlinx-coroutines-rx2 = { module = "org.jetbrains.kotlinx:kotlinx-coroutines-rx
kotlinx-coroutines-slf4j = { module = "org.jetbrains.kotlinx:kotlinx-coroutines-slf4j", version.ref = "kotlin-coroutines" }
kotlinx-coroutines-reactor = { module = "org.jetbrains.kotlinx:kotlinx-coroutines-reactor", version.ref = "kotlin-coroutines" }

log4j = { module = "org.apache.logging.log4j:log4j-core", version.ref = "log4j" }
log4j = { module = "org.apache.logging.log4j:log4j-core", version.ref = "managed-log4j" }
Copy link
Contributor

Choose a reason for hiding this comment

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

❌ This should be managed too, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@melix Yes, it's necessary. For example, if you want to use slf4j in your project in conjunction with micronaut-elasticsearch - for this you will need the 'org.apache.logging.log4j:log4j-to-slf4j:2.20.0' library and at the moment you will have to duplicate the log4j version in your project because this module is not in the bom file. We already discussed this here: #8138 . And now everything works fine with version 3.7.x. But these changes are lost in version 4.x

Copy link
Collaborator

Choose a reason for hiding this comment

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

@altro3 we moved versions outside of core intentionally in 4.0.x. I think micronaut-logging is probably the way to go.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@sdelamo good solution! But where is project micronaut-logging? :-)

@sdelamo
Copy link
Collaborator

sdelamo commented Mar 22, 2023

Closed via #8986

@sdelamo sdelamo closed this Mar 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants