[dev] Re: [cvs] commit: horde index.php

[Jan Schneider]

Zitat von Mike Cochrane <mike at graftonhall.co.nz>:

> > ----- Message from jan at horde.org ---------
> >     Date: Wed, 19 Feb 2003 05:03:53 -0800 (PST)
> >     From: Jan Schneider <jan at horde.org>
> > Reply-To: dev at lists.horde.org
> >  Subject: [cvs] commit: horde index.php
> >       To: cvs at lists.horde.org
> >
> > jan         2003/02/19 05:03:53 PST
> >
> >   Modified files:
> >     .                    index.php
> >   Log:
> >   Revert the last commit. It broke the url if the menu was disabled and
> no
> >   framset was used.
> >   If stripping the session id is necessary here, it was done in the
> wrong
> >   place.
> 
> Without stripping logging in is broken for all non cookies users. What is
> broken?
> 
> Removeing and adding the session id shouldn't break anything unless our
> removeparameter implementatation is broken.

For some reason the session id wasn't stripped from the url but Horde::url
applied to it. This resulted in an redirect url with &amp; instead of & as
the glue. This doesn't work with Location: headers.

Jan.

--
http://www.horde.org - The Horde Project
http://www.ammma.de - discover your knowledge
http://www.tip4all.de - Deine private Tippgemeinschaft