From 4a6598f102d8934c7502944a0addd400dd0cbcac Mon Sep 17 00:00:00 2001 From: Janis Date: Wed, 21 Jan 2026 11:45:12 +0100 Subject: [PATCH] feat: Change log level to warn for OpenID callback in OpenIDController --- knockoutwhistweb/app/controllers/OpenIDController.scala | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/knockoutwhistweb/app/controllers/OpenIDController.scala b/knockoutwhistweb/app/controllers/OpenIDController.scala index c9560ac..6f5fc90 100644 --- a/knockoutwhistweb/app/controllers/OpenIDController.scala +++ b/knockoutwhistweb/app/controllers/OpenIDController.scala @@ -61,6 +61,7 @@ class OpenIDController @Inject()( _ <- Option(sessionProvider.contains(provider)) authCode <- code } yield { + logger.warn(s"Authentication successful for $provider") openIDService.exchangeCodeForTokens(provider, authCode, sessionState.get).flatMap { case Some(tokenResponse) => openIDService.getUserInfo(provider, tokenResponse.accessToken).flatMap { @@ -68,7 +69,7 @@ class OpenIDController @Inject()( // Check if user already exists userManager.authenticateOpenID(provider, userInfo.id) match { case Some(user) => - logger.info(s"User ${userInfo.name} (${userInfo.id}) already exists, logging them in") + logger.warn(s"User ${userInfo.name} (${userInfo.id}) already exists, logging them in") // User already exists, log them in val sessionToken = sessionManager.createSession(user) Future.successful(Redirect(config.getOptional[String]("openid.mainRoute").getOrElse("/")) @@ -81,7 +82,7 @@ class OpenIDController @Inject()( )) .removingFromSession("oauth_state", "oauth_nonce", "oauth_provider", "oauth_access_token")) case None => - logger.info(s"User ${userInfo.name} (${userInfo.id}) not found, creating new user") + logger.warn(s"User ${userInfo.name} (${userInfo.id}) not found, creating new user") // New user, redirect to username selection Future.successful(Redirect(config.get[String]("openid.selectUserRoute")) .withSession(