PHPでログアウト機能を実装する

Question

PHP学習の一環としてログアウト機能を作りました。
まずセッションを開始させ、そのあとに、セッションクッキーの削除、そしてセッションの削除をし、最後にクッキーの削除をするプログラムを書きました(logout.php)
これでもログアウト機能は実装できたのですが、PHPマニュアルを見たところ自分の書いたコードと全然違っていました。マニュアルの方に従うべきなのはもちろんなのですが、自分のコードだと何が問題となるのでしょうか？ログイン時にクッキーに保存したのは、emailとpassword情報です。
参考にしたPHPのマニュアルです
https://www.php.net/manual/ja/function.session-destroy
logout.php

<?php 
session_start();
if(isset($_COOKIE['PHPSESSID'])){
setcookie("PHPSESSID","",time()-43200);
}
session_destroy();

setcookie("email","",time()-43200);
setcookie("password","",time()-43200);
header('Location:login.php');exit();
?>

Comments

[m.ts10806]

対象のPHPマニュアルのURLをご提示ください

[m.ts10806]

「PHPマニュアルと違うから問題がある」という質問でしょうか。
それとも「自分のコードに自信がある」前提での質問でしょうか。

今回の質問で得たいものを具体的に記載してください。

[退会済みユーザー]

PHPのマニュアルと違うから問題があるという質問です。
今回の質問で得たいものは自分のコードで問題はあるのか、またなぜ問題があるのであれば、マニュアルのように書かなければいけないのかということです。

[m.ts10806]

あと、本当にセキュリティ上の問題があるかどうか知りたいのでしたら、ログイン処理、ログアウトを呼び出している処理、含めて全て提示してください。単にセッションクリアしてるだけのコードでは判断できません。

[m.ts10806]

PHPマニュアルのコードはあくまでサンプルであり、目的を達成しているロジックであるかどうかのみを見るべきです。一字一句違わず実装しなさいという指示ではありません。

Answer 1

ログイン時にクッキーに保存したのは、emailとpassword情報です。

とあるので、そもそも何がしたいのかよくわかっていませんが、一般論として「ログアウト」に求める機能が実装できていれば良いと思います。

ログアウトに求める機能が記述されていないので、その定義から始めてください。

自身の中に定義がないようであれば、以下の記事が参考になるかと思います。
ログアウト機能の目的と実現方法

Answer 2

ログアウトなのですからsession_start()してsession_destroy()すればいいでしょう。
セッションから渡される値だけ信頼すればクッキーに書かれたemailやらpasswordは
特に必要ないと思います。