Hi Chris,
we'll find a solution, see my comments on WICKET-4260.
Sven
On 04/04/2012 04:44 PM, Chris Colman wrote:
Sven has tested this in IE and it worked. It seems it is some
additional specific which need to find and solve.
My quickstart has comments that specify the additional specific requirements to
make it happen - the app has to be in the context of a virtual host, not just
installed in the root 'webapps' directory. Has he tested it as a web host? I
gave a fairly thorough description of how to set it up to reproduce the error
in the JIRA comments.
The problem with plain 'logon' is that some users use something like:
'log:on', i.e. there is a colon in the first segment of the redirect
url. And here ALL browsers confuse that 'log:' is a scheme and they
try to find a URL handler that can process it. But most of the times
there is no such...
Using a colon ':' in a path sounds wrong - even though it may be legal. I
wouldn't think many people would do that and I've never seen it before.
I'm not sure if it's a good idea to break a fairly popular (though widely
hated) browser in the rare case that some users use a colon in the path.
Could the patch do a test to see if the path actually contains a ':' before it
does its funky business and if not, just leave the URL as is?
Or could the catering for ':' be a configurable option so that people who don't
put colons in their path could disable it?
On Wed, Apr 4, 2012 at 4:23 PM, Chris Colman
<[email protected]> wrote:
Ok, so if we go to
/landing
And user is not authenticated they get redirected to
./logon (a relative URL)
Which is effectively http://mydomain.com/./logon
Which IE can't handle but FF and Chrome seem to auto redirect to
http://mydomain.com/logon
It seems like we can't use the './' trick with IE. Wouldn't simply
'logon' without the leading './' also be a relative path? Why do we need to
prefix it with './'?
-----Original Message-----
From: Martin Grigorov [mailto:[email protected]]
Sent: Wednesday, 4 April 2012 10:38 PM
To: [email protected]
Subject: Re: IE redirect error with recent snapshot due to extra './'
Wicket works with relative paths only.
"/logon" is the mount path you specified in #mountPage() but later the
produced urls are relative to the filter path.
I.e. if a user is at /another/mount/point then a link to LoginPage
will look like: ../../logon
On Wed, Apr 4, 2012 at 2:33 PM, Chris Colman
<[email protected]> wrote:
The title of the bug report is interesting:
WICKET-4260 make sure relative urls start either with dot or dot-dot
I am redirecting to a page using its page class - which redirects to
the
mount for that page class - which is *always* an absolute path.
Eg., /logon
It seems strange that it would be treated as a relative path and so
have
a './' prefix added.
The redirection may be the same for all browsers but IE may not be
smart
enough to remove the redundant ./ from the URL.
-----Original Message-----
From: Martin Grigorov [mailto:[email protected]]
Sent: Wednesday, 4 April 2012 8:21 PM
To: [email protected]
Subject: Re: IE redirect error with recent snapshot due to extra './'
I gave you the ticket number in my first response :-)
On Wed, Apr 4, 2012 at 12:19 PM, Chris Colman
<[email protected]> wrote:
I've tried it on Tomcat 6.0.18, 6.0.20 and 6.0.35 and the problem
occurs
on all of them.
I remember seeing a recent Wicket git log entry about a change
involving
'./' and I'm wondering if it has something to do with that.
Chris
-----Original Message-----
From: Martin Grigorov [mailto:[email protected]]
Sent: Wednesday, 4 April 2012 5:48 PM
To: [email protected]
Subject: Re: IE redirect error with recent snapshot due to extra './'
Also can you tell us the version of Tomcat.
On Wed, Apr 4, 2012 at 9:45 AM, Martin Grigorov
<[email protected]>wrote:
Hi Chris,
Can you attach a quickstart to
https://issues.apache.org/jira/browse/WICKET-4260 ?
Thanks!
On Wed, Apr 4, 2012 at 9:42 AM, Chris Colman
<[email protected]
wrote:
** **
After upgrading to the latest Wicket snapshot a few days ago we
now
experience an IE specific problem when we redirect after
successful
log
on:
****
** **
It only happens for me on IE. On FireFox it works perfectly.****
** **
I've tracked it down to the 'continue to destination' that occurs
after
a
successful log in on the redirected authentication page.****
** **
On FireFox the 'continue' attempts to render:****
** **
http://acme1.acme.testurl.mydomain.com.au/myAdminPage****
** **
but on IE the redirect attempts to render:****
** **
http://acme1.acme.testurl.mydomain.com.au/./myAdminPage<http://acme1.
ac
me.t
esturl.mydomain.com.au/myAdminPage>
****
** **
Note the extra './' before myAdminPage****
** **
There is no page mounted at ./myAdminPage and so we get a Tomcat
error.**
**
** **
Note: this was verified on IE8 but may happen on other IE versions
also.*
***
** **
Yours sincerely,****
** **
Chris Colman****
****
Pagebloom Team Leader,****
Step Ahead Software
****
pagebloom - your business& your website growing together****
** **
**Sydney**: (+61 2) 9656 1278 ****Canberra****: (+61 2) 6100
2120
****
Email: [email protected]<//[email protected]>****
Website:****
http://www.pagebloom.com****
http://develop.stepaheadsoftware.com****
****
** **
--
Martin Grigorov
jWeekend
Training, Consulting, Development
http://jWeekend.com<http://jweekend.com/>
--
Martin Grigorov
jWeekend
Training, Consulting, Development
http://jWeekend.com<http://jweekend.com/>
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
--
Martin Grigorov
jWeekend
Training, Consulting, Development
http://jWeekend.com
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
--
Martin Grigorov
jWeekend
Training, Consulting, Development
http://jWeekend.com
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
--
Martin Grigorov
jWeekend
Training, Consulting, Development
http://jWeekend.com
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
