Merge pull request #43 from oxctl/reduce-logging-level

Reduce the logging level of the "normal" errors.
oxctl · Nov 23, 2023 · d49a710 · d49a710
2 parents 4f9d542 + 4ce14f7
commit d49a710
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/...ox/ctl/lti13/security/oauth2/client/lti/web/OAuth2AuthorizationRequestRedirectFilter.java b/...ox/ctl/lti13/security/oauth2/client/lti/web/OAuth2AuthorizationRequestRedirectFilter.java
@@ -205,14 +205,14 @@ private boolean hasNoPlatform(HttpServletRequest request) {
 	private void unsuccessfulRedirectForAuthorization(HttpServletRequest request, HttpServletResponse response,
                                                       Exception failed) throws IOException, ServletException {
 
-		if (logger.isErrorEnabled()) {
-			logger.error("Authorization Request failed: " + failed.toString(), failed);
-		}
 		if (failed instanceof InvalidInitiationRequestException) {
+			logger.info("Invalid initiation request: "+ failed);
 			response.sendError(HttpStatus.BAD_REQUEST.value(), failed.getMessage());
 		} else if (failed instanceof InvalidClientRegistrationIdException) {
+			logger.info("Invalid registration ID: "+ failed);
 			response.sendError(HttpStatus.NOT_FOUND.value(), failed.getMessage());
 		} else {
+			logger.error("Authorization Request failed: " + failed.toString(), failed);
 			response.sendError(HttpStatus.INTERNAL_SERVER_ERROR.value(), HttpStatus.INTERNAL_SERVER_ERROR.getReasonPhrase());
 		}
 	}