[WICKET-1864] MockHttpServletRequest does not support absolute redirection URLs.

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Cannot Reproduce
• Affects Version/s: 1.4-M3
• Component/s: wicket
• Assignee: Igor Vaynberg
• Reporter: Maarten Billemont
• Created: 08/Oct/08 12:00
• Updated: 20/Mar/16 15:30
• Resolved: 28/Aug/10 05:41

Description

Using the sendRedirect(String) method on an MockHttpServletRequest object causes bugged behaviour when the URL that is passed is absolute.

The following code needs to change (line 1277):
// We need to absolutize the redirect URL as we are not as smart as a web-browser
// (WICKET-702)
url = getContextPath() + getServletPath() + "/" + redirect;
into, for example:
// We need to absolutize the redirect URL as we are not as smart as a web-browser
// (WICKET-702)
url = redirect.charAt(0) == '/' ? redirect : redirect. getContextPath() + getServletPath() + "/" + redirect;

After a few redirects I end up at:
/BankApplication/BankApplication//BankApplication
Which is not where I want to be.