https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
--- Comment #25 from Jackie Rosen jackie.ro...@hushmail.com ---
*** Bug 260998 has been marked as a duplicate of this bug. ***
Seen from the domain http://volichat.com
Page where seen: http://volichat.com/adult-chat-rooms
Marked for
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
Mark Thomas ma...@apache.org changed:
What|Removed |Added
Status|NEEDINFO|RESOLVED
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
--- Comment #21 from Tino Kissig tino.kis...@exedio.com ---
At our current system we're getting the IllegalArgumentException when Tomcat
tries to normalize such a relative url for a redirect.
java.lang.IllegalArgumentException
at
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
Tino Kissig tino.kis...@exedio.com changed:
What|Removed |Added
CC|
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
--- Comment #22 from mgrigorov mgrigo...@apache.org ---
Wicket has a fix in versions 1.5.9 (not released yet) and 6.1.0.
Which version do you use ?
--
You are receiving this mail because:
You are the assignee for the bug.
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
Mark Thomas ma...@apache.org changed:
What|Removed |Added
Status|REOPENED|NEEDINFO
---
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
--- Comment #18 from mgrigorov mgrigo...@apache.org ---
(In reply to comment #16)
I have fixed the IAE in trunk and 7.0.x.
Unfortunately with this fix apps that work only with jsessionid encoded in the
url will still break. If JSESSIONID
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
--- Comment #19 from mgrigorov mgrigo...@apache.org ---
I think I found the solution. Wicket can pass always an absolute url to the web
container for encoding. Wicket knows the current base url and can make it
absolute safely. This way the
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
--- Comment #17 from wansho...@hotmail.com ---
(In reply to comment #16)
I have fixed the IAE in trunk and 7.0.x.
I am leaving this open while I wait for clarification from the Servlet EG as
to how relative URLs passed to encodeURL
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
--- Comment #16 from Mark Thomas ma...@apache.org ---
I have fixed the IAE in trunk and 7.0.x.
I am leaving this open while I wait for clarification from the Servlet EG as to
how relative URLs passed to encodeURL should be treated.
See
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
--- Comment #15 from papegaaij emond.papega...@topicus.nl ---
Ok, things are getting more clear to me now. Isn't it better to assume that
relative URLs always fall within the webapplication? The worst thing that could
happen, is that a
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
--- Comment #12 from papegaaij emond.papega...@topicus.nl ---
In my opinion, Tomcat should not convert relative URLs to absolute in
encodeURL. That should only be done in encodeRedirectURL. encodeURL can still
perform normalization, as long
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
--- Comment #13 from mgrigorov mgrigo...@apache.org ---
I agree with Emond.
By Servlet spec (actually the javadoc of
javax.servlet.http.HttpServletResponse#sendRedirect) only #sendRedirect()
should convert the url from relative to absolute.
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
--- Comment #14 from Mark Thomas ma...@apache.org ---
Folks, please re-read comment #11.
The output of encodeURL() is not and never will be normalized.
However, the Javadoc for encodeURL() allows/requires Tomcat to check if the
session
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
--- Comment #11 from Mark Thomas ma...@apache.org ---
OK. I see what the problem is here. It isn't quite as clear cut as comment #8
suggests.
To determine if the URL should be encoded, Tomcat attempts the following
checks:
a) is it not an
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
--- Comment #10 from papegaaij emond.papega...@topicus.nl ---
(In reply to comment #9)
Suppose you are at /a/b and you click a link to /a/b/c/d. This page contains
a link to /1/2. Wicket renders all urls relative, so the link to /1/2
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
--- Comment #9 from Mark Thomas ma...@apache.org ---
(In reply to comment #8)
The problem lies in the fact that encodeURL rewrites the url to absolute. I
think this should only be done in encodeRedirectURL.
That is a fair point. We'll
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
papegaaij emond.papega...@topicus.nl changed:
What|Removed |Added
Status|RESOLVED|REOPENED
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
papegaaij emond.papega...@topicus.nl changed:
What|Removed |Added
CC|
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
mgrigorov mgrigo...@apache.org changed:
What|Removed |Added
Status|RESOLVED|REOPENED
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
mgrigorov mgrigo...@apache.org changed:
What|Removed |Added
CC|
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
Mark Thomas ma...@apache.org changed:
What|Removed |Added
Status|REOPENED|RESOLVED
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
--- Comment #4 from mgrigorov mgrigo...@apache.org ---
OK, thanks!
In this case I'll let the reporter follow up with a test case that demonstrates
the problem with the urls provided in Wicket Jira.
--
You are receiving this mail because:
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
--- Comment #5 from Thyzz vonk.th...@gmail.com ---
Created attachment 29043
-- https://issues.apache.org/bugzilla/attachment.cgi?id=29043action=edit
Sample app source
According to Martin in
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
Thyzz vonk.th...@gmail.com changed:
What|Removed |Added
Status|RESOLVED|REOPENED
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
Mark Thomas ma...@apache.org changed:
What|Removed |Added
Status|REOPENED|RESOLVED
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
Mark Thomas ma...@apache.org changed:
What|Removed |Added
Status|NEW |RESOLVED
27 matches
Mail list logo