Can't we let it handle any protocol (contains("://") ?
As a side note, I'm sure there were already one or more similar jira
request (which we bumped to the next version to avoid changes between
RCs...), could you please check for them and link/close
appropriately ? thx.
g
On Nov 27, 2007, at 11:26 , [EMAIL PROTECTED] wrote:
Revision13140AuthorfgiustDate2007-11-27 11:26:00 +0100 (Tue, 27 Nov
2007)Log MessageMAGNOLIA-1891 don't add context path to absolute
urlsModified Paths
magnolia/trunk/magnolia-core/src/main/java/info/magnolia/cms/
filters/VirtualUriFilter.java
Diff
Modified: magnolia/trunk/magnolia-core/src/main/java/info/magnolia/
cms/filters/VirtualUriFilter.java (13139 => 13140)--- magnolia/
trunk/magnolia-core/src/main/java/info/magnolia/cms/filters/
VirtualUriFilter.java 2007-11-27 10:07:36 UTC (rev 13139) +++
magnolia/trunk/magnolia-core/src/main/java/info/magnolia/cms/
filters/VirtualUriFilter.java 2007-11-27 10:26:00 UTC (rev 13140)
@@ -70,8 +70,13 @@ if (targetUri.startsWith("redirect:")) { try { -
response.sendRedirect(request.getContextPath() - +
StringUtils.substringAfter(targetUri, "redirect:")); + String
redirectUrl = StringUtils.substringAfter(targetUri, "redirect:"); +
+ if (!redirectUrl.startsWith("http://";) && !redirectUrl.startsWith
("https://";)) { + redirectUrl = request.getContextPath() +
redirectUrl; + } + + response.sendRedirect(redirectUrl); return; }
catch (IOException e) {
----------------------------------------------------------------
for list details see
http://documentation.magnolia.info/docs/en/editor/stayupdated.html
----------------------------------------------------------------
