Browse Source

Return AuthorizationRequest from AuthorizationRequestRepository.removeAuthorizationRequest

Fixes gh-4652
pull/4232/merge
Joe Grandja 8 years ago
parent
commit
8a416793aa
  1. 2
      oauth2/oauth2-client/src/main/java/org/springframework/security/oauth2/client/web/AuthorizationRequestRepository.java
  2. 8
      oauth2/oauth2-client/src/main/java/org/springframework/security/oauth2/client/web/HttpSessionAuthorizationRequestRepository.java

2
oauth2/oauth2-client/src/main/java/org/springframework/security/oauth2/client/web/AuthorizationRequestRepository.java

@ -42,6 +42,6 @@ public interface AuthorizationRequestRepository { @@ -42,6 +42,6 @@ public interface AuthorizationRequestRepository {
void saveAuthorizationRequest(AuthorizationRequest authorizationRequest, HttpServletRequest request,
HttpServletResponse response);
void removeAuthorizationRequest(HttpServletRequest request);
AuthorizationRequest removeAuthorizationRequest(HttpServletRequest request);
}

8
oauth2/oauth2-client/src/main/java/org/springframework/security/oauth2/client/web/HttpSessionAuthorizationRequestRepository.java

@ -54,7 +54,11 @@ public final class HttpSessionAuthorizationRequestRepository implements Authoriz @@ -54,7 +54,11 @@ public final class HttpSessionAuthorizationRequestRepository implements Authoriz
}
@Override
public void removeAuthorizationRequest(HttpServletRequest request) {
request.getSession().removeAttribute(this.sessionAttributeName);
public AuthorizationRequest removeAuthorizationRequest(HttpServletRequest request) {
AuthorizationRequest authorizationRequest = this.loadAuthorizationRequest(request);
if (authorizationRequest != null) {
request.getSession().removeAttribute(this.sessionAttributeName);
}
return authorizationRequest;
}
}

Loading…
Cancel
Save