解决方案

Question

第一次尝试实施CSRF令牌......

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

网上的大多数内容都谈到通过_$POST php形式完成它......

所以...在阅读了所有内容之后，我已经推断出，因为我在内部部分有这个，所以我应该在那里设置令牌......

所以这就是我的所作所为......但显然我在这里误解了一些东西，因为它不起作用......不只是误解......我也不确定我是什么＆＃ 39;我应该......所以......如果你能解释我应该做的事情，我可以正确设置......我真的很感激

<?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;

EDITED

好的，所以..这是我到目前为止的工作原理：

<?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值，并且您在HTML表单的隐藏字段中输入的INPUT值是相同的值。

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']，在提交表单时 NOT ，您的代码当前在每次迭代时都会创建$_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();
}

CSRF内部$ _SERVER [＆＃34; REQUEST_METHOD＆＃34;]表单

1 个答案:

解决方案

编辑：

代码