UY-1439 fix 500 error code, switch og level in

OAuthTokenStatisticPublisher
unity-idm · Mar 12, 2024 · 68d3746 · 68d3746
1 parent 2ef32a9
commit 68d3746
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/oauth/src/main/java/pl/edu/icm/unity/oauth/as/token/access/OAuthTokenStatisticPublisher.java b/oauth/src/main/java/pl/edu/icm/unity/oauth/as/token/access/OAuthTokenStatisticPublisher.java
@@ -24,6 +24,7 @@
 import pl.edu.icm.unity.engine.api.authn.LoginSession;
 import pl.edu.icm.unity.engine.api.idp.statistic.IdpStatisticEvent;
 import pl.edu.icm.unity.exceptions.EngineException;
+import pl.edu.icm.unity.exceptions.InternalException;
 import pl.edu.icm.unity.oauth.as.OAuthASProperties;
 import pl.edu.icm.unity.oauth.as.OAuthRequestValidator;
 import pl.edu.icm.unity.oauth.as.OAuthSystemAttributesProvider;
@@ -72,7 +73,7 @@ void reportFailAsLoggedClient()
 		LoginSession loginSession = InvocationContext.getCurrent().getLoginSession();
 		if (loginSession == null)
 		{
-			log.error("Can not retrieve identity of the OAuth client, skippig error reporting");
+			log.debug("Can not retrieve identity of the OAuth client, skippig error reporting");
 			return;
 		}
 
@@ -84,7 +85,7 @@ void reportFailAsLoggedClient()
 
 		} catch (Exception e)
 		{
-			log.error("Can not retrieving identity of the OAuth client", e);
+			log.debug("Can not retrieving identity of the OAuth client", e);
 			return;
 		}
 
@@ -96,7 +97,7 @@ void reportFailAsLoggedClient()
 			clientName = getClientName(clientEntity);
 		} catch (Exception e)
 		{
-			log.error("Can not retrieving client name attribute of the OAuth client", e);
+			log.debug("Can not retrieving client name attribute of the OAuth client", e);
 			return;
 		}
 
@@ -112,8 +113,7 @@ private String getClientName(EntityParam clientEntity)
 			attrs = unsecureAttributesMan.getAllAttributes(clientEntity, true, oauthGroup, null, false);
 		} catch (EngineException e)
 		{
-			log.error("Problem retrieving attributes of the OAuth client", e);
-			throw new InternalError("Internal error, can not retrieve OAuth client's data");
+			throw new InternalException("Internal error, can not retrieve OAuth client's data", e);
 		}
 
 		Optional<AttributeExt> clientNameAttr = attrs.stream()
@@ -140,7 +140,7 @@ void reportSuccess(String clientUsername, String clientName, EntityParam owner)
 					new ApplicationId(clientUsername), Instant.now());
 		} catch (EngineException e)
 		{
-			log.error("Can not set last access attribute", e);
+			log.debug("Can not set last access attribute", e);
 		}
 
 	}
@@ -175,7 +175,7 @@ private Endpoint getEndpoint()
 			}
 		} catch (Exception e)
 		{
-			log.error("Can not get relateed OAauth authz endpoint for token endpoint " + endpoint.getName(), e);
+			log.debug("Can not get relateed OAauth authz endpoint for token endpoint " + endpoint.getName(), e);
 			return endpoint.getEndpoint();
 		}
 	}