CommonContexts icon indicating copy to clipboard operation
CommonContexts copied to clipboard

Fix checks for relative redirects

Open aderuwe opened this issue 12 years ago • 4 comments

Maybe it's me doing something wrong, but I think calling locatePath() on the expected URL breaks checks on relative redirects.

aderuwe avatar Apr 29 '13 10:04 aderuwe

In which case is it broken ?

stof avatar Apr 30 '13 22:04 stof

When $header is a relative redirect to /some/url then calling locatePath() on the expected one prepends the base_url configuration option, making the match fail. Maybe the solution is to instead call locatePath() on $header as well?

aderuwe avatar May 02 '13 09:05 aderuwe

Ping @stof

Does my comment make sense or am I doing something wrong?

aderuwe avatar May 13 '13 09:05 aderuwe

:+1: failing here too

ricoli avatar Jun 18 '13 15:06 ricoli