search

atomikos / transactions-essentials

Development repository for next major release of
https://www.atomikos.com/Main/TransactionsEssentials
Other
461 stars 139 forks source link

LoggerFactory should check for SLF4J API instead of implementation. #202

Open rob-valor opened 9 months ago

rob-valor commented 9 months ago

Describe the bug LoggerFactory should check for SLF4J API (org.slf4j.Logger) instead of the implementation (org.slf4j.impl.StaticLoggerBinder), just like it is done for Log4J2

Additional context The issue now is that a message is incorrectly printed to the console, altough SLF4J is on the classpath and the com.atomikos.logging.Slf4jLogger instance is not used, but is should be.

I understand that this project is not intended for support - because bug reports may or may not be considered for inclusion some day (in a future release). If this is issue is important to me then I can go to https://www.atomikos.com/Main/SupportOverview and arrange a paid support subscription.

pascalleclercq commented 3 months ago

In that case, we may select sl4j logging framework without having any implementation in the classpath ? Do I miss something ?