let all loggers use non-anonymous parent class names

CHANGELOG.md

@@ -1,5 +1,6 @@
 # Unreleased
 * core: logger(name) is now accessible without a class instance context
+* core: Any.logger() will now take the closest non-anonymous superclass
 * server: errors.on<SomeException>(StatusCode) convenience reified function added
 * jdbc: BaseCrudRepository.list() and by() now both have the suffix parameter
 * jdbc: handle Postgres "cached plan must not change result type" exception by enabling autosave=conservative by default

core/src/Logger.kt

@@ -4,7 +4,14 @@ package klite
 import java.lang.System.Logger.Level.*
 
 fun logger(name: String): System.Logger = System.getLogger(name)
-fun Any.logger(): System.Logger = System.getLogger(javaClass.name)
+fun Any.logger(): System.Logger = logger(nonAnonymousClassName())
+
+internal fun Any.nonAnonymousClassName(): String {
+  var cls = javaClass
+  while (cls.isAnonymousClass) cls = cls.superclass
+  return cls.name
+}
+
 inline fun System.Logger.debug(msg: String) = log(DEBUG, msg)
 inline fun System.Logger.info(msg: String) = log(INFO, msg)
 inline fun System.Logger.warn(msg: String) = log(WARNING, msg)

core/test/LoggerTest.kt

@@ -0,0 +1,18 @@
+package klite
+
+import ch.tutteli.atrium.api.fluent.en_GB.toBeAnInstanceOf
+import ch.tutteli.atrium.api.fluent.en_GB.toEqual
+import ch.tutteli.atrium.api.verbs.expect
+import org.junit.jupiter.api.Test
+import java.util.Date
+
+class LoggerTest {
+  @Test fun name() {
+    expect(nonAnonymousClassName()).toEqual(LoggerTest::class.java.name)
+    expect(object: Date() {}.nonAnonymousClassName()).toEqual(Date::class.java.name)
+  }
+
+  @Test fun testLogger() {
+    expect(logger()).toBeAnInstanceOf<System.Logger>()
+  }
+}

server/src/klite/ErrorHandler.kt

@@ -12,7 +12,7 @@ open class BusinessException(messageKey: String, cause: Throwable? = null): Exce
 typealias ThrowableHandler<T> = HttpExchange.(e: T) -> ErrorResponse?
 
 open class ErrorHandler {
-  private val log = logger(ErrorHandler::class.java.name)
+  private val log = logger()
   private val handlers = mutableMapOf<KClass<out Throwable>, ThrowableHandler<Throwable>>()
   private val statusCodes = mutableMapOf<KClass<out Throwable>, StatusCode>(
     IllegalArgumentException::class to BadRequest,