You are viewing a plain text version of this content. The canonical link for it is here.
Posted to pluto-scm@portals.apache.org by dd...@apache.org on 2004/12/22 05:26:04 UTC

svn commit: r123043 - /portals/pluto/trunk/container/src/java/org/apache/pluto/core/impl/ActionResponseImpl.java

Author: ddewolf
Date: Tue Dec 21 20:26:02 2004
New Revision: 123043

URL: http://svn.apache.org/viewcvs?view=rev&rev=123043
Log:
Applying patch for PLUTO-91.  Submitted by David Jencks.  Thanks
Modified:
   portals/pluto/trunk/container/src/java/org/apache/pluto/core/impl/ActionResponseImpl.java

Modified: portals/pluto/trunk/container/src/java/org/apache/pluto/core/impl/ActionResponseImpl.java
Url: http://svn.apache.org/viewcvs/portals/pluto/trunk/container/src/java/org/apache/pluto/core/impl/ActionResponseImpl.java?view=diff&rev=123043&p1=portals/pluto/trunk/container/src/java/org/apache/pluto/core/impl/ActionResponseImpl.java&r1=123042&p2=portals/pluto/trunk/container/src/java/org/apache/pluto/core/impl/ActionResponseImpl.java&r2=123043
==============================================================================
--- portals/pluto/trunk/container/src/java/org/apache/pluto/core/impl/ActionResponseImpl.java	(original)
+++ portals/pluto/trunk/container/src/java/org/apache/pluto/core/impl/ActionResponseImpl.java	Tue Dec 21 20:26:02 2004
@@ -116,8 +116,10 @@
                 ResourceURLProvider provider = InformationProviderAccess.getDynamicProvider(getHttpServletRequest()).getResourceURLProvider(getInternalPortletWindow());
                 if (location.indexOf("://") != -1) {
                     provider.setAbsoluteURL(location);
-                } else {
+                } else if (location.startsWith("/")) {
                     provider.setFullPath(location);
+                } else {
+                    throw new IllegalArgumentException("Only absolute and full path URLs are allowed.  The relative path '" + location+"' is not valid.");
                 }
                 location = redirectResponse.encodeRedirectURL(provider.toString());
                 //redirectResponse.sendRedirect(location);