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

Old Flyway dependency cannot auto detect SLF4J 2.0 #470

Closed
huyi0912 opened this issue Oct 20, 2023 · 0 comments · Fixed by #471
Closed

Old Flyway dependency cannot auto detect SLF4J 2.0 #470

huyi0912 opened this issue Oct 20, 2023 · 0 comments · Fixed by #471
Assignees
Labels
type: bug Something isn't working

Comments

@huyi0912
Copy link

huyi0912 commented Oct 20, 2023

Expected Behavior

Logging output should respect on log level setting.
For example add following to logback.xml should print out all sql executed
<logger name="org.flywaydb.core.internal.sqlscript" level="DEBUG">
<appender-ref ref="STDOUT"/>
</logger>

Actual Behaviour

Setting log level to debug but always got info level only

Steps To Reproduce

By following guide "ACCESS A DATABASE WITH MICRONAUT DATA R2DBC". When I try to see the logging for real sql executed by flyway. Found no matter how detail level I setup.Always only info level was logged.After some digging find flyway have a bug detecting SLF4j 2.0(flyway/flyway#3730). And it's been fixed at Flyway 9.22.2 (2023-09-21).But latest micronaut depends on older 9.21.2. Temporary solution is add dependency(Maven)
<dependency>
<groupId>org.flywaydb</groupId>
<artifactId>flyway-core</artifactId>
<version>9.22.3</version>
</dependency>

Environment Information

Mac Sonoma 14
JDK Oracle GraalVM 21+35.1

Example Application

No response

Version

4.1.4,4.1.5

@sdelamo sdelamo added this to 4.1.6 Oct 20, 2023
@sdelamo sdelamo added the type: bug Something isn't working label Oct 20, 2023
@wetted wetted moved this to In Progress in 4.1.6 Oct 23, 2023
@wetted wetted moved this from In Progress to Ready for Review in 4.1.6 Oct 23, 2023
@github-project-automation github-project-automation bot moved this from Ready for Review to Done in 4.1.6 Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants