libre.sh/keycloak-scim
2
0
Fork 0
Code Issues 14 Pull requests Releases Packages Activity Actions 6

Errors are now reported using sendError instead of setStatus.

Browse source 
This change was made to enable the use of error pages defined in web.xml.
This commit is contained in:
Thomas Raehalme 2015-11-06 10:48:36 +02:00
parent 7b7fbd3257
commit 68edf9ce48
2 changed files with 4 additions and 20 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
integration/adapter-core/src/main/java/org/keycloak/adapters/AuthenticatedActionsHandler.java
View file

@ -57,7 +57,7 @@ public class AuthenticatedActionsHandler {
protected boolean abortTokenResponse() {
if (facade.getSecurityContext() == null) {
log.debugv("Not logged in, sending back 401: {0}",facade.getRequest().getURI());
facade.getResponse().setStatus(401);
facade.getResponse().sendError(401);
facade.getResponse().end();
return true;
}
@ -94,7 +94,7 @@ public class AuthenticatedActionsHandler {
log.debugv("allowedOrigins did not contain origin");
}
facade.getResponse().setStatus(403);
facade.getResponse().sendError(403);
facade.getResponse().end();
return true;
}

20
integration/adapter-core/src/main/java/org/keycloak/adapters/OAuthRequestAuthenticator.java
View file

@ -174,23 +174,7 @@ public class OAuthRequestAuthenticator {
final String state = getStateCode();
final String redirect = getRedirectUri(state);
if (redirect == null) {
return new AuthChallenge() {
@Override
public boolean challenge(HttpFacade exchange) {
exchange.getResponse().setStatus(403);
return true;
}
@Override
public boolean errorPage() {
return true;
}
@Override
public int getResponseCode() {
return 403;
}
};
return challenge(403);
}
return new AuthChallenge() {
@ -283,7 +267,7 @@ public class OAuthRequestAuthenticator {
@Override
public boolean challenge(HttpFacade exchange) {
exchange.getResponse().setStatus(code);
exchange.getResponse().sendError(code);
return true;
}
};