From 4255957e2da3e42b93304c81fab011b31b08d68b Mon Sep 17 00:00:00 2001
From: srosse <stephane.rosse@frentix.com>
Date: Tue, 16 Feb 2021 07:12:29 +0100
Subject: [PATCH] no-jira: reduce error message to info
---
.../java/org/olat/login/oauth/spi/OpenIdConnectApi.java | 8 ++++----
.../login/oauth/spi/OpenIdConnectFullConfigurableApi.java | 8 ++++----
2 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/src/main/java/org/olat/login/oauth/spi/OpenIdConnectApi.java b/src/main/java/org/olat/login/oauth/spi/OpenIdConnectApi.java
index bdb0bd5457e..77017acbaba 100644
--- a/src/main/java/org/olat/login/oauth/spi/OpenIdConnectApi.java
+++ b/src/main/java/org/olat/login/oauth/spi/OpenIdConnectApi.java
@@ -103,21 +103,21 @@ public class OpenIdConnectApi extends DefaultApi20 {
if(!oauthModule.getOpenIdConnectIFIssuer().equals(idJson.get("iss"))
|| !oauthModule.getOpenIdConnectIFIssuer().equals(accessJson.get("iss"))) {
allOk &= false;
- log.error("iss don't match issuer");
+ log.info("iss don't match issuer");
}
if(!getApiKey().equals(idJson.get("aud"))) {
allOk &= false;
- log.error("aud don't match application key");
+ log.info("aud don't match application key");
}
if(!oVerifier.getState().equals(oVerifier.getSessionState())) {
allOk &= false;
- log.error("state doesn't match session state");
+ log.info("state doesn't match session state");
}
if(!oVerifier.getSessionNonce().equals(idJson.get("nonce"))) {
allOk &= false;
- log.error("session nonce don't match verifier nonce");
+ log.info("session nonce don't match verifier nonce");
}
return allOk ? new OAuth2AccessToken(idToken, oVerifier.getState()) : null;
diff --git a/src/main/java/org/olat/login/oauth/spi/OpenIdConnectFullConfigurableApi.java b/src/main/java/org/olat/login/oauth/spi/OpenIdConnectFullConfigurableApi.java
index c20e6cb592d..37bd356eeb4 100644
--- a/src/main/java/org/olat/login/oauth/spi/OpenIdConnectFullConfigurableApi.java
+++ b/src/main/java/org/olat/login/oauth/spi/OpenIdConnectFullConfigurableApi.java
@@ -100,22 +100,22 @@ public class OpenIdConnectFullConfigurableApi extends DefaultApi20 {
if(!provider.getIssuer().equals(idJson.get("iss"))
|| !provider.getIssuer().equals(accessJson.get("iss"))) {
allOk &= false;
- log.error("iss don't match issuer");
+ log.info("iss don't match issuer");
}
if(!provider.getAppKey().equals(idJson.get("aud"))) {
allOk &= false;
- log.error("aud don't match application key");
+ log.info("aud don't match application key");
}
if(!oVerifier.getState().equals(oVerifier.getSessionState())) {
allOk &= false;
- log.error("state doesn't match session state");
+ log.info("state doesn't match session state");
}
if(!oVerifier.getSessionNonce().equals(idJson.get("nonce"))) {
allOk &= false;
- log.error("session nonce don't match verifier nonce");
+ log.info("session nonce don't match verifier nonce");
}
return allOk ? new OAuth2AccessToken(idToken, oVerifier.getState()) : null;
} catch (JSONException e) {
--
GitLab