From e9cb3d2b42123bc2e394e571d5cddc650084cc90 Mon Sep 17 00:00:00 2001
From: eikek <eike.kettner@posteo.de>
Date: Thu, 7 Jul 2022 10:55:30 +0200
Subject: [PATCH] Better logging in oidc flows

---
 .../src/main/scala/docspell/oidc/CodeFlow.scala   | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/modules/oidc/src/main/scala/docspell/oidc/CodeFlow.scala b/modules/oidc/src/main/scala/docspell/oidc/CodeFlow.scala
index c1a01995..ff5c06a2 100644
--- a/modules/oidc/src/main/scala/docspell/oidc/CodeFlow.scala
+++ b/modules/oidc/src/main/scala/docspell/oidc/CodeFlow.scala
@@ -111,15 +111,19 @@ object CodeFlow {
           token <- r.attemptAs[AccessToken].value
           _ <- token match {
             case Right(t) =>
-              logger.trace(s"Got token response: $t")
+              logger.trace(s"Got token response (status=${r.status.code}): $t")
             case Left(err) =>
-              logger.error(err)(s"Error decoding access token: ${err.getMessage}")
+              logger.error(err)(
+                s"Error decoding access token (status=${r.status.code}): ${err.getMessage}"
+              )
           }
         } yield token.toOption
       case r =>
-        logger
-          .error(s"Error obtaining access token '${r.status.code}' / ${r.as[String]}")
-          .map(_ => None)
+        for {
+          body <- r.bodyText.compile.string
+          _ <- logger
+            .error(s"Error obtaining access token status=${r.status.code}, body=$body")
+        } yield None
     })
   }
 
@@ -177,5 +181,4 @@ object CodeFlow {
       logAction = Some((msg: String) => logger.trace(msg))
     )(c)
   }
-
 }