内部$ _SERVER [“ REQUEST_METHOD”]表单的CSRF

Question

尝试首次实现CSRF令牌...

我有一个内部的$_SERVER["REQUEST_METHOD"]表单...

在线上的大多数东西都谈论通过_$POST php表单完成它...

所以...阅读完所有内容后，我认为由于我已将其放在内部，所以我应该在那里设置令牌...

所以这就是我所做的...但是显然我在这里误解了某件事，因为它不起作用...不仅仅是误解...我也不知道我该怎么做...所以...如果您能解释我应该怎么做，那么我可以正确设置它...我将不胜感激

<?php
session_start(); //allows use of session variables

  $token = md5(uniqid(rand(), TRUE));
  $_SESSION['token'] = $token;
  $_SESSION['token_time'] = time();


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

   if (empty($_POST["first-name"])) {
     $firstNameErr = "First name is required";
   } else {
     $first_name = test_input($_POST["first-name"]);
   }

    if (empty($_POST["last-name"])) {
     $lastNameErr = "Last name is required";
   } else {
     $last_name = test_input($_POST["last-name"]);
   }

   if (empty($_POST["email"])) {
     $emailErr = "Email is required";
   } else {
     $email = test_input($_POST["email"]);
   }

   if (empty($_POST["message"])) {
     $messageErr = "Message is required";
   } else {
     $message = test_input($_POST["message"]);
   }


   if(isset($first_name) && isset($last_name) && isset($email) && isset($message) && isset($token))
   {
     $_SESSION['first_name'] = $first_name;
     $_SESSION['last_name'] = $last_name;
     $_SESSION['email'] = $email;
     $_SESSION['message'] = $message;
     $_SESSION['token'] = $token;
     header("Location: SessionsCRSF.php");
   }
}

function test_input($data) {
   $data = trim($data);
   $data = stripslashes($data);
   $data = htmlspecialchars($data);
   return $data;
}
?>

表单在同一页上（放置摘录，但您明白了）：

<form class="ui form"  method="post" action="<?php echo htmlspecialchars($_SERVER["PHP_SELF"]);?>">

  <input type="hidden" name="token" value="<?php echo $token; ?>" />

  <div class="field">
    <label>First Name</label>
    <input name="first-name" id="first-name" placeholder="First Name" type="text" value="<?php if(isset($first_name)) { echo $first_name; }?>">
     <?php if(isset($firstNameErr)) print ('<span class="error">* ' . $firstNameErr . '</span>'); ?>
  </div>

(...)

</form>

但是我不知道如何通过标题中列出的SessionsCRSF.php表单对其进行测试...

这是SessionsCRSF.php表单的代码段，因为其余只是PHPMailer的内容：

<?php

session_start();

    if ($_POST['token'] == $_SESSION['token'])
    {
      die('working, yay?'); 
    }


$first_name = $_SESSION['first-name'];
$last_name = $_SESSION['last-name'];
$email = $_SESSION['email'];
$message = nl2br($_SESSION['message']);

require 'PHPMailerAutoload.php';

$mail = new PHPMailer;

已编辑

好的，所以..这是我到目前为止有效的方法：

<?php
session_start(); //allows use of session variables

if ($_SERVER["REQUEST_METHOD"] == "POST" && $_SESSION['token'] == $_POST['token'] && !empty($_SESSION['token'])) {

   if (empty($_POST["first-name"])) {
     $firstNameErr = "First name is required";
   } else {
     $first_name = test_input($_POST["first-name"]);
   }

    if (empty($_POST["last-name"])) {
     $lastNameErr = "Last name is required";
   } else {
     $last_name = test_input($_POST["last-name"]);
   }

   if (empty($_POST["email"])) {
     $emailErr = "Email is required";
   } else {
     $email = test_input($_POST["email"]);
   }

   if (empty($_POST["message"])) {
     $messageErr = "Message is required";
   } else {
     $message = test_input($_POST["message"]);
   }

   if(isset($first_name) && isset($last_name) && isset($email) && isset($message))
   {
     $_SESSION['first_name'] = $first_name;
     $_SESSION['last_name'] = $last_name;
     $_SESSION['email'] = $email;
     $_SESSION['message'] = $message;
     header("Location: contact9SessionsCRSF2.php");
     exit; 
   }
}
else {
  $token = md5(uniqid(rand(), TRUE));
  $_SESSION['token'] = $token;
  $_SESSION['token_time'] = time();
}

function test_input($data) {
   $data = trim($data);
   $data = stripslashes($data);
   $data = htmlspecialchars($data);
   return $data;
}
?>

Answer 1

您的页面包含$_SESSION值，并且您的INPUT值在HTML表单的隐藏字段中具有相同的值。

Page 1:
[SESSION] ... [INPUT]

然后，您将表单提交到目标地址（此处为第2页），然后检查所提交的隐藏字段值是否等于$_SESSION值，该值未被输入表单字段传递。

此输入值可从$_POST数组中找到。

Page 2: 
[SESSION] ... [POST]

具体针对您的问题，您将表单提交到同一页面的PHP_SELF页面（通常， PHP_SELF是一个不好的变量，因此不鼓励使用）。 但是，对数据的检查是在sessionCRSF.php页面上完成的，因此您应该做的是更新表单以放入：

<form action='sessionCRSF.php' ... > 

然后，在该页面的顶部，对令牌值的检查应该成功。

---

您似乎对SERVER请求方法有些困惑，有GET / POST / PUT和其他方法，如果您有一个

我有一个内部的$ _SERVER [“ REQUEST_METHOD”]表单

这到底是什么？ 据我所知，如果是表单，则它是GET / POST方法，并且不是内部的-即使它在同一服务器上，也从一页到另一页，因此适用上述概念。

解

通过讨论和澄清，我现在可以看到解决方案：

在处理表单提交的If语句中，您没有对令牌发布值$_POST['token']进行任何处理。 因此，您需要添加一条语句以最简单的形式在设置$_SESSION表单数据的值的代码点处添加一行：

$_SESSION['posted_token'] = $_POST['token'];

这就是您所需要的，然后在sessionCRSF.php比较这两个值，然后得到：

 if ($_SESSION['posted_token'] == $_SESSION['token'])
    {
      die('working, yay?'); 
    }

（您之前让我对“内部”表格等感到困惑！）

编辑：

您需要在生成表单时设置原始的$_SESSION['token'] ，而不是在提交表单时设置，当前您的代码在每次迭代时都会创建$_SESSION['token']值，并且由于页面自身引用了它表示令牌值-随机-永远不会相同。 你需要设置

if (form submitted){
   save submitted POSTED token
}
else{
    /// form not submitted
    generate a token and save to session
} 

不要在同一迭代中同时做这两个事情！！

码

因为有时候看它会更容易....

代码已更新，可以比较POST提交时的令牌，因此无需将数据发送到另一个要比较的页面。

<?php
session_start(); //allows use of session variables


if ($_SERVER["REQUEST_METHOD"] == "POST" && 
$_SESSION['token'] == $_POST['token'] && !empty($_SESSION['token'])) {

   if (empty($_POST["first-name"])) {
     $firstNameErr = "First name is required";
   } else {
     $first_name = test_input($_POST["first-name"]);
   }

    ...etc...

   if(isset($first_name) && isset($last_name) && isset($email) && isset($message))
   {
     $_SESSION['first_name'] = $first_name;
     $_SESSION['last_name'] = $last_name;
     $_SESSION['email'] = $email;
     $_SESSION['message'] = $message;

     header("Location: SessionsCRSF.php");
     exit; //ALWAYS end execution after sending location headers. 
   }
}
else {
  $token = md5(uniqid(rand(), TRUE));
  $_SESSION['token'] = $token;
  $_SESSION['token_time'] = time();
}