mirror/keycloak
1
0
Fork 0
mirror of https://github.com/keycloak/keycloak.git synced 2025-12-16 20:15:46 -06:00
Code Issues Packages Projects Releases Wiki Activity

fix: preventing raw stacktrace response and error log (#44815)

Browse Source 
closes: #44712

Signed-off-by: Steve Hawkins <shawkins@redhat.com>
This commit is contained in:
Steven Hawkins
2025-12-16 09:28:29 -05:00
committed by GitHub
parent 40eb51f10c
commit 5bf740e383
2 changed files with 13 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
services/src/main/java/org/keycloak/services/error/KeycloakErrorHandler.java
View File

@@ -107,7 +107,7 @@ public class KeycloakErrorHandler implements ExceptionMapper<Throwable> {
}
return Response.status(responseStatus)
.header(HttpHeaders.CONTENT_TYPE, jakarta.ws.rs.core.MediaType.APPLICATION_JSON_TYPE.toString())
.type(MediaType.APPLICATION_JSON_TYPE)
.entity(error)
.build();
}

15
services/src/main/java/org/keycloak/utils/MediaTypeMatcher.java
View File

@@ -3,8 +3,12 @@ package org.keycloak.utils;
import jakarta.ws.rs.core.HttpHeaders;
import jakarta.ws.rs.core.MediaType;
import org.jboss.logging.Logger;
public class MediaTypeMatcher {
private static final Logger logger = Logger.getLogger(MediaTypeMatcher.class);
public static boolean isHtmlRequest(HttpHeaders headers) {
return isAcceptMediaType(headers, MediaType.TEXT_HTML_TYPE);
}
@@ -14,10 +18,15 @@ public class MediaTypeMatcher {
}
private static boolean isAcceptMediaType(HttpHeaders headers, MediaType textHtmlType) {
for (MediaType m : headers.getAcceptableMediaTypes()) {
if (!m.isWildcardType() && m.isCompatible(textHtmlType)) {
return true;
try {
for (MediaType m : headers.getAcceptableMediaTypes()) {
if (!m.isWildcardType() && m.isCompatible(textHtmlType)) {
return true;
}
}
} catch (Exception e) {
// illegal state, or illegal argument are possible
logger.debug("Could not determine if the media type is accepted", e);
}
return false;
}