[PHP] header

[Verwijderd]

hi,

nu ben ik wel iets heel vaags tegen gekomen. direct maar de code:

PHP:

session_start();
if ($_REQUEST['Logout']) {
    $_SESSION = array();
    session_destroy();
}
if ($_REQUEST['Login']) {
    include 'db.php';
    $username = $_REQUEST['username'];
    $password = $_REQUEST['password'];
    $query = mysql_query("SELECT ...");
    if (mysql_num_rows($query) == 1) {
        session_register('username');
        $_SESSION['username'] = $username;
    }
}
header("location: http://www.domein.nl/".$origin);

da's dus een in- en uitlog script. maar als ik inlog (dus de 2e 'if' doorloop) gaat-ie gewoon naar de juiste url, maar als ik uitlog niet (Forbidden - You don't have permission to access to this document on this server.) is dat geen typo trouwens?! "to access to this document", zal wel aan de tijd liggen..

ik dacht, misschien ligt het aan de 2 regels om de sessie te destroyen (dat-ie dan alle variabelen destroyed ofzo) maar als ik die weglaat gebeurt er hetzelfde

ik heb zelfs de if's eens omgedraaid (ja ik moest er zelf ook om lachen )

anyone?

edit:

euh.. de titel is niet helemaal overgekomen..

[ Voor 11% gewijzigd door Verwijderd op 31-12-2003 03:37 ]

[Stamgastje]

Uhm, ben niet echt bekend met PHP, maar heeft dat niet te maken met "session_destroy"? Ik bedoel dus dat $origin dan niet meer de waarde heeft die hij na inloggen wel heeft (is immers destroyed).

[ Voor 41% gewijzigd door Stamgastje op 31-12-2003 04:05 ]

[Terranca]

PHP:

header("location: http://www.domein.nl/".$origin);

Waar haal je die $origin vandaan Als je die uit de session haalt verklaart t meteen een hoop nml

[edit + ps]
Je code bevat wel overbodige dingen. session_register('username') is niet nodig als je hem vervolgens set met $_SESSION['username'] = $username. Volgens mij is je $_SESSION = Array () ook niet nodig als je session_destroy () oproept.

[ Voor 44% gewijzigd door Terranca op 31-12-2003 04:08 ]

[Verwijderd]

• het ligt niet aan session_destroy() (zie TS)
• $origin is geen session variabele
• $origin komt van de (zelfde) vorige pagina (form)

nou ja, ik heb het wel opgelost door login en logout in 2 verschillende .php files te behandelen, maar dat was wel wat omslachtig.. tnx anyway!

[edit]
bedankt voor de opmerkingen, de session_register was inderdaad overbodig en

PHP:

if ($_REQUEST['Logout']) {
    $_SESSION = array();
    session_destroy();
}

heb ik gewoon maar netjes van php.net overgenomen.

[ Voor 34% gewijzigd door Verwijderd op 31-12-2003 11:50 ]

[Verwijderd]

wat me nu opvalt is dat ik altijd netjes al mijn variabelen uit een form ophaal met

PHP:

foreach($HTTP_POST_VARS AS $key => $value) {
    $$key = stripslashes($value);
}

maar alles wat je 'POST' met een html-form is dus automatisch al een variabele in PHP (zie $origin in TS). met de code in deze post wordt dan alleen alles nog even gestripped. zie ik dat goed? is deze foreach-lus verder overbodig?

[_Sunnyboy_]

welke waarde heeft $origin eigenlijk na het uitloggen. Heb je al geprobeerd om

PHP:

header("location: http://www.domein.nl/".$origin);

te vervangen door

PHP:

echo("location: http://www.domein.nl/".$origin);

Dan kan je zien wat ie eigenlijk probeert uit te voeren.

[Verwijderd]

sorry folks! heb de fout uit de TS gevonden, echt te dom.. *schaam* in het form om uit te loggen was ik de hidden field 'origin' vergeten (uiteraard). het was gisteravond gewoon te laat.. excuse me

[Alex]

Dat houdt dus in dat je het field wel moet aanroepen als $_REQUEST['origin']

[ Voor 10% gewijzigd door Alex op 31-12-2003 13:20 ]

[Verwijderd]

nou, dat dacht ik ook, maar hij doet het dus zonder $_REQUEST ook.. (code is nu zoals TS)

[Alex]

Dat blijft het ranzig, de ene keer wel GLOBALS gebruiken en de andere keer niet...

[Verwijderd]

Inderdaad, ik zou register_globals uitzetten (ik maak hieruit op dat je hem aan hebt staan) zodat je wel $_POST, $_GET of $_REQUEST etc. moet gebruiken ...

Het is nog veiliger om register_globals uit te hebben ook...

[Verwijderd]

aha, ik heb dat ff nagelezen hier en hier en mijn host maar eens even geinformeerd

het staat namelijk automatisch uit vanaf PHP 4.2.0, dus is er vast een oudere versie geinstalleerd

bedankt voor de info

[Room42]

offtopic:
versie kun je checken met phpinfo();

[creative8500]

PHP:

if (version_compare (phpversion (), '4.2.0'))
{
    // code ...
}
else die ('To run this script you need PHP 4.2.0 or a newer version.');

of:

PHP:

if (!version_compare ('4.2.0', phpversion ())
    die ('To run this script you need PHP 4.2.0 or a newer version.');

[hobbit_be]

voor al dat soort zaken gebruik ik gewoon global functions zoals:

gGetSessionVariable()
gGetInputVariable(); //eerst post dan get

etc.

Zodoende kan de volgende versie van PHP niet WEER eens alles overhoop gooien. Doe ik al heel lang en zit nu toch een paar versies verder zonder ooit een probleem te zijn tegengekomen (behalve de call-by-ref natuurlijk).

Overal waar een INI of versie iets uitmaakt zet ik het in een functie. En als je dan een compiling template engine gebruikt heb je helemaal geen overhead.

[Verwijderd]

ok tnx, ik zal nog eens kijken

Dit zag ik trouwens bij phpMyAdmin 2.2.6 staan:

Furthermore, we encountered crashes of the php module / binary when working with PHP 4.2.0 (tested with PHP 4.2.0 RC1 to RC4) and trying to download a (zipped) dump file using MS Internet Explorer. That is why we recommend not to run phpMyAdmin with PHP 4.2.0.

edit:

maar we zitten ondertussen al bij versie 4.3.4 alleen toevallig dat het weer precies versie 4.2.0 was

[ Voor 15% gewijzigd door Verwijderd op 31-12-2003 15:58 ]

[Verwijderd]

uhmmm..

nu staat er dus toch een nieuwere versie van PHP op mijn host: PHP Version 4.3.1

scroll.. scroll.. en inderdaad: register_globals staat On.

voegt weinig toe aan dit topic, maar ik wou het toch even kwijt