cocoon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 32762] - [PATCH] flow redirector should allow explicit 'cocoon:' scheme
Date Fri, 21 Oct 2005 16:21:27 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=32762>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32762





------- Additional Comments From mlundquist2@comcast.net  2005-10-21 18:21 -------
It's not really "complicated URL-checking machinery" :-).  We currently check to see if the
URL contains 
a colon, and all my patch does is to skip that check if the URL starts with "cocoon:/".

Your suggestion is similar to Jörg's in Comment #4, and the reason IMHO it is not good is
because the 
newbie who doesn't understand cocoon.sendPage() will try to use it for an external redirect,
and will get 
a less-than-helpful error message, e.g. "Resource Not Found: cocoon:/http://foobar".  My patch

preserves the current check and helpful error message.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

Mime
View raw message