Fix #565 - [logger-f-logback-mdc-monix3] is not compatible with logback 1.5.17 due to changes in slf4j 2.0.17

- Update slf4j to 2.0.17, logback to 1.5.17 and logback-scala-interop to 1.17.0
- Add slf4j MDC support to set MDCAdapter in MDC

Author: kevin-lee

build.sbt

@@ -81,6 +81,8 @@ lazy val loggerF = (project in file("."))
 //    sbtLoggingJs,
     catsJvm,
 //    catsJs,
+    slf4jMdcJvm,
+//    slf4jMdcJs,
     logbackMdcMonix3Jvm,
 //    logbackMdcMonix3Js,
     testKitJvm,
@@ -253,6 +255,24 @@ lazy val cats    =
 lazy val catsJvm = cats.jvm
 lazy val catsJs  = cats.js
 
+lazy val slf4jMdc    = module(ProjectName("slf4j-mdc"), crossProject(JVMPlatform, JSPlatform))
+  .settings(
+    description := "Logger for F[_] - A tool to set MDC's MDCAdapter",
+    libraryDependencies ++= Seq(
+      libs.slf4jApi,
+      libs.cats % Test,
+    ) ++ libs.tests.hedgehogLibs,
+    libraryDependencies := libraryDependenciesRemoveScala3Incompatible(
+      scalaVersion.value,
+      libraryDependencies.value,
+    ),
+  )
+  .dependsOn(
+    core
+  )
+lazy val slf4jMdcJvm = slf4jMdc.jvm
+lazy val slf4jMdcJs  = slf4jMdc.js
+
 lazy val logbackMdcMonix3    = module(ProjectName("logback-mdc-monix3"), crossProject(JVMPlatform, JSPlatform))
   .settings(
     description := "Logger for F[_] - logback MDC context map support for Monix 3",
@@ -270,6 +290,7 @@ lazy val logbackMdcMonix3    = module(ProjectName("logback-mdc-monix3"), crossPr
   )
   .dependsOn(
     core,
+    slf4jMdc,
     monix       % Test,
     slf4jLogger % Test,
   )
@@ -554,14 +575,14 @@ lazy val props =
 
     final val ExtrasVersion = "0.25.0"
 
-    final val Slf4JVersion   = "2.0.16"
-    final val LogbackVersion = "1.5.16"
+    final val Slf4JVersion   = "2.0.17"
+    final val LogbackVersion = "1.5.17"
 
     final val Log4sVersion = "1.10.0"
 
     final val Log4JVersion = "2.19.0"
 
-    val LogbackScalaInteropVersion = "1.16.0"
+    val LogbackScalaInteropVersion = "1.17.0"
   }
 
 lazy val libs =

modules/logger-f-logback-mdc-monix3/shared/src/main/scala/loggerf/logger/logback/Monix3MdcAdapter.scala

@@ -3,7 +3,7 @@ package loggerf.logger.logback
 import ch.qos.logback.classic.LoggerContext
 import logback_scala_interop.JLoggerFMdcAdapter
 import monix.execution.misc.Local
-import org.slf4j.{LoggerFactory, MDC}
+import org.slf4j.LoggerFactory
 
 import java.util.{Map => JMap, Set => JSet}
 import scala.jdk.CollectionConverters._
@@ -45,10 +45,7 @@ trait Monix3MdcAdapterOps {
 
   @SuppressWarnings(Array("org.wartremover.warts.Null"))
   protected def initialize0(monix3MdcAdapter: Monix3MdcAdapter): Monix3MdcAdapter = {
-    val field = classOf[MDC].getDeclaredField("mdcAdapter")
-    field.setAccessible(true)
-    field.set(null, monix3MdcAdapter) // scalafix:ok DisableSyntax.null
-    field.setAccessible(false)
+    org.slf4j.SetMdcAdapter(monix3MdcAdapter)
     monix3MdcAdapter
   }
 

modules/logger-f-logback-mdc-monix3/shared/src/test/scala/loggerf/logger/logback/Monix3MdcAdapterSpec.scala

@@ -4,10 +4,14 @@ import cats.syntax.all._
 import hedgehog._
 import hedgehog.runner._
 import monix.eval.{Task, TaskLocal}
-import org.slf4j.MDC
+import org.slf4j.spi.MDCAdapter
+import org.slf4j.{MDC, SetMdcAdapter}
 
+import java.io.{PrintWriter, StringWriter}
 import java.time.Instant
+import java.util
 import scala.jdk.CollectionConverters._
+import scala.util.control.NonFatal
 
 /** @author Kevin Lee
   * @since 2023-07-03
@@ -20,7 +24,23 @@ object Monix3MdcAdapterSpec extends Properties {
    */
   implicit val opts: Task.Options = Task.defaultOptions.enableLocalContextPropagation
 
-  private val monixMdcAdapter: Monix3MdcAdapter = Monix3MdcAdapter.initialize()
+  @SuppressWarnings(Array("org.wartremover.warts.Throw"))
+  private val monixMdcAdapter: Monix3MdcAdapter =
+    try {
+      Monix3MdcAdapter.initialize()
+    } catch {
+      case NonFatal(ex) =>
+        val writer = new StringWriter()
+        val out    = new PrintWriter(writer)
+        ex.printStackTrace(out)
+        System
+          .err
+          .println(
+            s"""Error when initializing Monix3MdcAdapter: ${writer.toString}
+           |""".stripMargin
+          )
+        throw ex // scalafix:ok DisableSyntax.throw
+    }
 
   override def tests: List[Test] = List(
     property("Task - MDC should be able to put and get a value", testPutAndGet),
@@ -39,6 +59,7 @@ object Monix3MdcAdapterSpec extends Properties {
     property("Task - MDC: It should return context map for getCopyOfContextMap", testGetCopyOfContextMap),
     property("Task - MDC: It should return context map for getPropertyMap", testGetPropertyMap),
     property("Task - MDC: It should return context map for getKeys", testGetKeys),
+    property("test SetMdcAdapter", testSetMdcAdapter),
   )
 
   def before(): Unit =
@@ -484,4 +505,48 @@ object Monix3MdcAdapterSpec extends Properties {
     }.toMap
   }
 
+  @SuppressWarnings(Array("org.wartremover.warts.Equals"))
+  def testSetMdcAdapter: Property = for {
+    mdcAdapterName1 <- Gen.string(Gen.ascii, Range.linear(5, 5)).log("mdcAdapterName1")
+    mdcAdapterName2 <- Gen.string(Gen.ascii, Range.linear(5, 5)).log("mdcAdapterName2")
+    mdcAdapterName  <- Gen.constant(mdcAdapterName1 + mdcAdapterName2).log("mdcAdapterName")
+  } yield {
+    val mdcAdapter = TestMdcAdapter(mdcAdapterName)
+    val before     = MDC.getMDCAdapter()
+
+    SetMdcAdapter(mdcAdapter)
+
+    val after = MDC.getMDCAdapter()
+
+    Result.all(
+      List(
+        Result.diffNamed("before should not be equal to the new MDCAdapter", before, mdcAdapter)(_ != _),
+        (after ==== mdcAdapter).log("after SetMdcAdapter should be equal to the new MDCAdapter"),
+      )
+    )
+  }
+
+  @SuppressWarnings(Array("org.wartremover.warts.Var"))
+  final case class TestMdcAdapter(name: String) extends MDCAdapter {
+
+    override def put(key: String, `val`: String): Unit = ???
+
+    override def get(key: String): String = ???
+
+    override def remove(key: String): Unit = ???
+
+    override def clear(): Unit = ???
+
+    override def getCopyOfContextMap: util.Map[String, String] = ???
+
+    override def setContextMap(contextMap: util.Map[String, String]): Unit = ???
+
+    override def pushByKey(key: String, value: String): Unit = ???
+
+    override def popByKey(key: String): String = ???
+
+    override def getCopyOfDequeByKey(key: String): util.Deque[String] = ???
+
+    override def clearDequeByKey(key: String): Unit = ???
+  }
 }

modules/logger-f-slf4j-mdc/shared/src/main/scala/org/slf4j/SetMdcAdapter.scala

@@ -0,0 +1,16 @@
+package org.slf4j
+
+import org.slf4j.spi.MDCAdapter
+
+/** @author Kevin Lee
+  * @since 2025-03-09
+  */
+trait SetMdcAdapter {
+  def apply(mdcAdapter: MDCAdapter): Unit
+}
+
+object SetMdcAdapter {
+  def apply(mdcAdapter: MDCAdapter): Unit = {
+    MDC.setMDCAdapter(mdcAdapter)
+  }
+}