[php] sessie slaat niet op

zondag 27 juni 2010 11:21

Acties:

0 Henk 'm!

Topicstarter

Ik snap niet wat ik fout doe of welke instelling er niet goed staat. In index.php check ik de inloggegevens van een gebruiker zodra deze kloppen leg ik deze vast is een sessie en redirect ik de gebruiker naar editWishList.php. Bij editWishList.php check ik of er in de sessie een gebruik zit. Zit deze er niet in dan redirect ik de gebruiker naar index.php. De grap is dat ik in geen waardes lijk te hebben in mijn sessie zodra ik uitkom op editWishList.php. Wat gaat er niet niet?

Index.php

PHP:

<?php
require_once("Includes/db.php");

$logonSuccess = true;

if ($_SERVER["REQUEST_METHOD"] == "POST") {

    if (WishDB::getInstance()->verify_wisher_credentials($_POST["user"], $_POST["userpassword"]) == 1) {
        session_start();
        $_SESSION["user"] = $_POST["user"];
        header('Location: editWishList.php');
    } else {
        $logonSuccess = false;
    }
}
?>
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
    <head>
        <title>Title</title>
        <meta http-equiv="content-type" content="text/html; charset=UTF-8">
    </head>
    <body>
        <form name="logon" action="index.php" method="POST" >
            Username: <input type="text" name="user"/>
            Password  <input type="password" name="userpassword"/>
            <?php
            if (!$logonSuccess)
                echo "Invalid name and/or password";
            ?>
            <input type="submit" value="Edit My Wish List"/>
        </form>
    </body>
</html>

editWishList.php

PHP:

<?php
session_start();
if (array_key_exists("user", $_SESSION)) {
    echo "Hello " . $_SESSION["user"];
} else {
   header('Location: index.php');
   exit;
}
?>

zondag 27 juni 2010 11:24

Acties:

0 Henk 'm!

Spookie

Bij index.php moet ook

session_start();

zondag 27 juni 2010 11:25

Acties:

0 Henk 'm!

DaRKie

Moet je in index.php ook niet session_start() aanroepen?

zondag 27 juni 2010 11:27

Acties:

0 Henk 'm!

durkp

In index.php

Als je daar eens

code:

1	session_start();

bovenaan zet.

zondag 27 juni 2010 11:35

Acties:

0 Henk 'm!

Liqued

Topicstarter

op regel 9 roep ik session_start al aan. Het verplaatsen naar regel 2 van session_start heeft geen effect helaas.

zondag 27 juni 2010 11:37

Acties:

0 Henk 'm!

durkp

Komen de $_POST wel goed binnen?

Heb je al nagekeken, als dat gewoon werkt?

zondag 27 juni 2010 11:38

Acties:

0 Henk 'm!

console

Liqued schreef op zondag 27 juni 2010 @ 11:35:
op regel 9 roep ik session_start al aan. Het verplaatsen naar regel 2 van session_start heeft geen effect helaas.

Doe eens een print_r( $_SESSION ); en kijk of er uberhaupt wel iets geset word?

zondag 27 juni 2010 11:39

Acties:

0 Henk 'm!

joostvanpinxten

Niet heel erg nuttig commentaar hierboven... Je hebt in de index.php maar een zeer minimale sessie, maar dat is hier helemaal geen probleem.

Ik heb bovenstaande code genomen en uitgevoerd (natuurlijk zonder de DB controle, die heb ik er even uitgestript) en dan krijg ik toch daadwerkelijk een sessie met daarin de index user ingesteld. editWishList.php doet het ook gewoon goed.

Welke versie van PHP gebruik je en op welk platform? Probeer het volgende eens:

PHP:

<?php
session_start();
if(empty($_SESSION['increment'])){
echo $_SESSION['increment'] = 1;
}else{
echo ++$_SESSION['increment'];
}

?>

Elke refresh moet dan een hoger getal opleveren en dan weet je zeker dat je sessies correct werken.

Verder is het werken met een debugger (zoals bv NuSphere PhpEd die heeft) in een geval als deze erg handig!

[ Voor 0% gewijzigd door joostvanpinxten op 27-06-2010 11:42 . Reden: ++ voor de var ipv erna ]

zondag 27 juni 2010 11:43

Acties:

0 Henk 'm!

Liqued

Topicstarter

Ik heb het script van joostvanpinxten uitgevoerd. De sessie waarde blijft op 1 staan. Op http://database.orange-dragons.com/temp.php kun je de phpinfo zien.

zondag 27 juni 2010 11:46

Acties:

0 Henk 'm!

orf

Zorg eens dat display_errors aan staat en zet eens bovenaan je script error_reporting(E_ALL);
Dan zie je wellicht wat er niet goed gaat.

zondag 27 juni 2010 11:50

Acties:

0 Henk 'm!

Lye

Heb je al geprobeerd om error_reporting op E_STRICT te zetten? Waarschijnlijk wordt je sessie niet eens gestart dus zal daar al een fout zitten.

zondag 27 juni 2010 11:54

Acties:

0 Henk 'm!

joostvanpinxten

session.save_path is het enige dat van mijn php-configuratie (onder het kopje session dan) verschilt, hier staat bij jou /var/lib/php/session

Als je deze server zelf opgezet hebt, dan kan het zijn dat de php binary geen rechten heeft om in dat mapje te schrijven. Ik zou toch verwachten dat dat een error is, die door PHP gegooid wordt, maar voor de zekerheid even het volgende bovenaan zetten.

PHP:

<?php
ini_set('display_errors',true);
error_reporting(E_ALL | E_STRICT);
?>

@Lyle en @orf zonder display errors zie je dan alsnog niets.... en het lijkt erop dat dit een productie server is....

Verder zal je misschien wat E_NOTICE's krijgen omdat je bepaalde array-indices zomaar probeert aan te spreken, zonder dat deze geset zijn.

Overigens vind ik de empty() (of bijna equivalent de isset()) manier leesbaarder dan je constructie met array_key_exists()

[ Voor 23% gewijzigd door joostvanpinxten op 27-06-2010 11:58 . Reden: E_NOTICE verwachting.... ]

zondag 27 juni 2010 11:54

Acties:

0 Henk 'm!

Liqued

Topicstarter

op http://database.orange-dragons.com/temp2.php heb ik nu het volgende script staan

PHP:

<?php
ini_set('display_errors',true); 
error_reporting(E_ALL | E_STRICT
session_start();
if (empty($_SESSION['increment'])) {
    echo $_SESSION['increment'] = 1;
} else {
    echo++$_SESSION['increment'];
}
?>

geen foutmeldingen

--edit------------------------------------------------------------
ik heb hem nu aangepast met de input van joostvanpinxten en tataaaa... we hebben een foutmelding. ik ga gelijk de rechten checken. ik denk dat het komt omdat ik nu php draai in cgi modus. eens even checken hoe ik dat moet fixen.

[ Voor 35% gewijzigd door Liqued op 27-06-2010 11:58 ]

zondag 27 juni 2010 11:59

Acties:

0 Henk 'm!

orf

joostvanpinxten schreef op zondag 27 juni 2010 @ 11:54:
@Lyle en @orf zonder display errors zie je dan alsnog niets.... en het lijkt erop dat dit een productie server is....

Lees mijn eerste (van 2) zinnen nog eens?

zondag 27 juni 2010 12:02

Acties:

0 Henk 'm!

joostvanpinxten

@orf je hebt gelijk, scuzie, te snel gelezen!

zondag 27 juni 2010 12:06

Acties:

0 Henk 'm!

r0b

Even http://php.net/manual/en/function.session-save-path.php aanpassen.

zondag 27 juni 2010 12:07

Acties:

0 Henk 'm!

Lye

joostvanpinxten schreef op zondag 27 juni 2010 @ 11:54:
Verder zal je misschien wat E_NOTICE's krijgen omdat je bepaalde array-indices zomaar probeert aan te spreken, zonder dat deze geset zijn.

Overigens vind ik de empty() (of bijna equivalent de isset()) manier leesbaarder dan je constructie met array_key_exists()

empty() gebruiken voor het controleren of array indices bestaan is absoluut niet aan te raden. Als de waarde die hoort bij dat element toevallig gelijk is aan 0, "0", false, null, array of een gedeclareerde class property zonder waarde krijg je dus foute resultaten. Isset daarentegen is wel te gebruiken, maar houd rekening met null-waarden.

zondag 27 juni 2010 12:09

Acties:

0 Henk 'm!

Liqued

Topicstarter

problem solved. ik heb de rechten van de sessie dir aangepast aan de "nieuwe" owner (aangezien ik geswitch ben van "apache" als gebruiker naar de cgi gebruiker).

ps. is dit een veilige manier?

Onderwerpen