Hi there,

I'd like you to confirm that the following is actually an issue.

In 
StateAwareResponseImpl<http://svn.apache.org/viewvc/portals/pluto/trunk/pluto-container/src/main/java/org/apache/pluto/internal/impl/StateAwareResponseImpl.java?view=markup>
#sendRedirect, the ResourceURLProvider is updated with the absolute or
relative URL of the redirect address. Then the URL is encoded and stored in
the variable location, and this is checked:

if (location.indexOf("/") == -1)
  throw new IllegalArgumentException("There is a relative path given, an
IllegalArgumentException must be thrown.");

However, I think it should be:

if (location.indexOf("*://*") == -1)
  throw new IllegalArgumentException("There is a relative path given, an
IllegalArgumentException must be thrown.");

Because relative URLs, such as "foo/bar", can have the character "/", so the
exception wouldn't be thrown for them.

Is that correct? How should I proceed to fix it?

Thanks,

Pablo

Reply via email to