Cookie löschen funktioniert nicht

Stef · 25. September 2018

Hey,

ich erstelle bei einem Login ein Cookie. Nun wenn der User Logout klickt soll dem sein Cookie gelöscht werden. Jedoch wird dieser nie gelöscht. Egal was ich mache.
Ich habe bereits sehr viel gegoogelt. Dies konnte aber nicht mein Problem lösen.

So erstelle ich den Cookie:

PHP

if($verify){
       setcookie("username", $userData->username, time() + 3600);
       
}

So möchte ich ihn löschen:

PHP

//Class
public function deleteCookie(){
    setcookie("username", "" , time() - 10000);
}
//

if(isset($_POST["logout"])){
    $logout->deleteCookie();
}

Die Funktion gibt immer True zurück und trotzdem wird der Cookie nicht gelöscht.

Was mache ich falsch?

Grüße,

Stef

m.scatello · 25. September 2018

Zitat von Stef

Die Funktion gibt immer True zurück

Welche Funktion? Ich sehe nicht, wo da ein Return-Wert existiert.

basti1012 · 25. September 2018

Ich kann da nicht viel zu sagen ,aber zur Info bei mir geht es so

PHP

<?php
class test
{
    public function deleteCookie(){
            setcookie("username",'', time() + 3600);
               return "Cookie ist weg";
    }
}

if(isset($_POST["logout"])){
    $logout = new test;
    echo $logout->deleteCookie();
} 
 if(isset($_POST["erstelle"])){
      setcookie("username",'Bastis test cookie', time() + 3600);
      echo "cookie erstellet";
}
?>
<form action="cookie.php" method="post">
<input type="submit" name="logout" value="Cookie löschen">
</form>
<form action="cookie.php" method="post">
<input type="submit" name="erstelle" value="Cookie erstellen">
</form>

Alles anzeigen

Stef · 25. September 2018

Hey,

Zitat von m.scatello

Welche Funktion? Ich sehe nicht, wo da ein Return-Wert existiert.

Diese Funktion in der Klasse:

PHP

//Class
public function deleteCookie(){
    setcookie("username", "" , time() - 10000);
}
//

Das return habe ich nach dem testen wieder entfernt.

Die richtige Klasse sieht so aus:

PHP

class Logout{

    private $cookieName;

    public function __construct($cookieName){
        $this->cookieName = $cookieName;
    }

    public function deleteCookie(){
        setcookie($this->cookieName, "" , time() - 3600);
    }

    public function goToIndexPage(){
        header("LOCATION: ../index.php");
    }

    
}

Alles anzeigen

So wird es ausgeführt:

PHP

<?php
    

    $username = $_COOKIE["username"];

    if(!isset($username) || empty($username)){
        exit();
    }

    require_once("../classes/logout.class.php");

    $logout = new Logout("username");

    if(isset($_POST["logout"])){
        $logout->deleteCookie();
        $logout->goToIndexPage();
    }

?>

<!DOCTYPE html>
<html>
<head>
    <meta charset="utf-8">
    <title>Home</title>
</head>
<body>
    <header>
        <h1>Willkommen <?= $username; ?></h1>
        <form method="post">
            <button name="logout">Ausloggen</button>
        </form>
        
    </header>
</body>
</html>

Alles anzeigen

Auf der Index Page frage ich dann ab ob Cookie Username gesetzt ist und wenn ja lass ich eine Verlinkung zur Home-Page ausgeben.

Diese wird immer ausgegeben.

Das ist die Index Page:

PHP

<?php


require_once("classes/login.class.php");
require_once("classes/database.class.php");

// Password = 1234;

$username = "Hans";
$password = "1234";


$login = new Login($username, $password);
$login->getUserFromDatabase();

$userData = $login->getUserData();

if(isset($_POST["verify"])){

    $verify = $login->verifyUser();


    if($verify){
        setcookie("username", $userData->username, time() + 3600);
        header("LOCATION: sites/home.php");
    }

}


if(isset($_COOKIE["username"])){
    echo "<a href='sites/home.php'>Home</a>";
}



?>

<form method="post">
    <button name="verify">Verify</button>
</form>

Alles anzeigen

basti1012 :

Wenn du bei time() ein + anhängst dann sagst du damit aus, dass der Cookie 1 Stunde laufen soll. So löschst du diesen ja nicht. Löschen kann man den laut meiner Recherchen nur wenn man time() - Zahl nimmt. Dann liegt der Cookie in der Vergangenheit und ist somit abgelaufen.

Selbst das nützt bei mir nichts.

Grüße,

Stef

m.scatello · 25. September 2018

Bitte mal nachdenken, was daran nicht in Ordnung ist:

Code

$username = $_COOKIE["username"];
if(!isset($username) || empty($username)){
        exit();
}

Und hier

Code

$logout = new Logout("username");
if(isset($_POST["logout"])){
        $logout->deleteCookie();
        $logout->goToIndexPage();
}

musst du mir mal erklären, warum du ein new machst, wenn $_POST["logout"] nicht gesetzt ist.

Und ein hoch gedrehtes Error-Reporting sehe ich auch nicht.

Stef · 26. September 2018

Hey,

Zitat von m.scatello
Bitte mal nachdenken, was daran nicht in Ordnung ist:
Code
$username = $_COOKIE["username"];
if(!isset($username) || empty($username)){
        exit();
}

Was soll da denn nicht in Ordnung sein?

Zitat
Und hier
Code
$logout = new Logout("username");
if(isset($_POST["logout"])){        
        $logout->deleteCookie();        
        $logout->goToIndexPage();
}
musst du mir mal erklären, warum du ein new machst, wenn $_POST["logout"] nicht gesetzt ist.

Und ein hoch gedrehtes Error-Reporting sehe ich auch nicht.

Ich habe es einfach so gemacht. Die Klasse wird nun im if-block instanziert.

Error-Reporting habe ich bereits angeschaltet. Aber es kommt keine Meldung von diesem.

Weiterhin besteht das Problem.

Grüße,

Stef

m.scatello · 26. September 2018

Zitat von Stef

Hey,

Was soll da denn nicht in Ordnung sein?

Du greifst auf $_COOKIE["username"], egal ob die Variable gesetzt ist oder nicht.

Und deshalb glaube ich auch nicht, dass du das Error-Reporting hochgedreht hast, denn sonst würdest du ein Meldung bekommen, wenn der Keks nicht gesetzt ist.

Zitat

Ich habe es einfach so gemacht. Die Klasse wird nun im if-block instanziert.

Wenn du die Klasse nur in dem if-Block brauchst, dann mache auch in diesem das new!

Stef · 26. September 2018

Hey,

ich habe die home-Seite noch nicht via Browser so geöffnet. Danke.

Das alles ändert aber nichts daran, dass der Cookie gelöscht wird. Der Cookie wird einfach nicht gelöscht. Obwohl die Function ausgeführt wird.

Auch wenn die Applikation so aussieht:

PHP: home.php

if(isset($_POST["logout"])){

        $logout = new Logout("username");

        if($logout->deleteCookie()){
            $logout->goToIndexPage();
        }
        
    }

PHP: Logout Class

public function deleteCookie(){
        
        if(setcookie($this->cookieName, "" , time() - 3600)){
            return true;
        } else{
            return false;
        }
    }

wird die Funktion goToIndexPage ausgeführt. Das heißt der Cookie sollte nicht mehr existieren. Aber das tut dieser immernoch. Der wird einfach nicht gelöscht.

Warum denn ?

Grüße,

Stef

m.scatello · 27. September 2018

Ich würde dir ja gerne helfen, befürchte aber, dass mein Beitrag deaktiviert werden könnte und daher eine Antwort relativ sinnlos wäre!

Stef · 27. September 2018

Hey,

Zitat von m.scatello

Ich würde dir ja gerne helfen, befürchte aber, dass mein Beitrag deaktiviert werden könnte und daher eine Antwort relativ sinnlos wäre!

Wenn Beiträge freundlich verfasst sind dann werden auch keine deaktiviert. Habe ich dir auch im Willkommensthema mitteilen wollen.

Cookie löschen funktioniert nicht

Jetzt mitmachen!

Teilen