Renamed obtainFullRequestUrl to obtainFullSavedRequestUrl.
This commit is contained in:
parent
28d4fa4665
commit
96dd564b79
|
@ -265,7 +265,7 @@ public abstract class AbstractProcessingFilter extends SpringSecurityFilter impl
|
||||||
chain.doFilter(request, response);
|
chain.doFilter(request, response);
|
||||||
}
|
}
|
||||||
|
|
||||||
public static String obtainFullRequestUrl(HttpServletRequest request) {
|
public static String obtainFullSavedRequestUrl(HttpServletRequest request) {
|
||||||
HttpSession session = request.getSession(false);
|
HttpSession session = request.getSession(false);
|
||||||
|
|
||||||
if (session == null) {
|
if (session == null) {
|
||||||
|
@ -418,7 +418,7 @@ public abstract class AbstractProcessingFilter extends SpringSecurityFilter impl
|
||||||
protected String determineTargetUrl(HttpServletRequest request) {
|
protected String determineTargetUrl(HttpServletRequest request) {
|
||||||
// Don't attempt to obtain the url from the saved request if
|
// Don't attempt to obtain the url from the saved request if
|
||||||
// alwaysUsedefaultTargetUrl is set
|
// alwaysUsedefaultTargetUrl is set
|
||||||
String targetUrl = alwaysUseDefaultTargetUrl ? null : obtainFullRequestUrl(request);
|
String targetUrl = alwaysUseDefaultTargetUrl ? null : obtainFullSavedRequestUrl(request);
|
||||||
|
|
||||||
if (targetUrl == null) {
|
if (targetUrl == null) {
|
||||||
targetUrl = getDefaultTargetUrl();
|
targetUrl = getDefaultTargetUrl();
|
||||||
|
|
|
@ -91,7 +91,7 @@ public class ExceptionTranslationFilterTests extends TestCase {
|
||||||
filter.doFilter(request, response, chain);
|
filter.doFilter(request, response, chain);
|
||||||
assertEquals("/mycontext/login.jsp", response.getRedirectedUrl());
|
assertEquals("/mycontext/login.jsp", response.getRedirectedUrl());
|
||||||
assertEquals("http://www.example.com/mycontext/secure/page.html", AbstractProcessingFilter
|
assertEquals("http://www.example.com/mycontext/secure/page.html", AbstractProcessingFilter
|
||||||
.obtainFullRequestUrl(request));
|
.obtainFullSavedRequestUrl(request));
|
||||||
}
|
}
|
||||||
|
|
||||||
public void testAccessDeniedWhenNonAnonymous() throws Exception {
|
public void testAccessDeniedWhenNonAnonymous() throws Exception {
|
||||||
|
@ -185,7 +185,7 @@ public class ExceptionTranslationFilterTests extends TestCase {
|
||||||
filter.doFilter(request, response, chain);
|
filter.doFilter(request, response, chain);
|
||||||
assertEquals("/mycontext/login.jsp", response.getRedirectedUrl());
|
assertEquals("/mycontext/login.jsp", response.getRedirectedUrl());
|
||||||
assertEquals("http://www.example.com/mycontext/secure/page.html", AbstractProcessingFilter
|
assertEquals("http://www.example.com/mycontext/secure/page.html", AbstractProcessingFilter
|
||||||
.obtainFullRequestUrl(request));
|
.obtainFullSavedRequestUrl(request));
|
||||||
}
|
}
|
||||||
|
|
||||||
public void testRedirectedToLoginFormAndSessionShowsOriginalTargetWithExoticPortWhenAuthenticationException()
|
public void testRedirectedToLoginFormAndSessionShowsOriginalTargetWithExoticPortWhenAuthenticationException()
|
||||||
|
@ -217,7 +217,7 @@ public class ExceptionTranslationFilterTests extends TestCase {
|
||||||
filter.doFilter(request, response, chain);
|
filter.doFilter(request, response, chain);
|
||||||
assertEquals("/mycontext/login.jsp", response.getRedirectedUrl());
|
assertEquals("/mycontext/login.jsp", response.getRedirectedUrl());
|
||||||
assertEquals("http://www.example.com:8080/mycontext/secure/page.html", AbstractProcessingFilter
|
assertEquals("http://www.example.com:8080/mycontext/secure/page.html", AbstractProcessingFilter
|
||||||
.obtainFullRequestUrl(request));
|
.obtainFullSavedRequestUrl(request));
|
||||||
}
|
}
|
||||||
|
|
||||||
public void testStartupDetectsMissingAuthenticationEntryPoint() throws Exception {
|
public void testStartupDetectsMissingAuthenticationEntryPoint() throws Exception {
|
||||||
|
|
Loading…
Reference in New Issue