【问题标题】:Is this php script secure to use in my site?这个 php 脚本在我的网站中使用是否安全?
【发布时间】:2017-02-17 11:28:18
【问题描述】:

我想用密码保护我网站中的一些内容,我正在考虑使用这个 php 脚本

您认为这是一个好方法吗?

您是否知道更好的方法来完成这项任务或改进(如果需要)瘦的方法?

从数据库加载内容的代码是:

<?php


error_reporting(0);
include("config.php");


if (!isset($_REQUEST["p"])) {

    echo 'document.write("<div id=\"protected_'.intval($_REQUEST["id"]).'\">");';
    echo 'document.write("<form onsubmit=\'return LoadContent(\"'.intval($_REQUEST["id"]).'\",\"protected_'.intval($_REQUEST["id"]).'\",document.getElementById(\"pass_'.intval($_REQUEST["id"]).'\").value); return false;\'\"><input type=\'password\' size=\'30\' placeholder=\'Content is protected! Enter password.\' id=\"pass_'.intval($_REQUEST["id"]).'\"></form>");';
    echo 'document.write("</div>");';

} else {

    $sql = "SELECT * FROM ".$SETTINGS["data_table"]." WHERE `id`='".intval($_REQUEST["id"])."' AND password='".mysql_real_escape_string($_REQUEST["p"])."'";
    $sql_result = mysql_query ($sql, $connection ) or die ('request "Could not execute SQL query" '.$sql);

    if (mysql_num_rows($sql_result)==1) {
        $row = mysql_fetch_assoc($sql_result);
        echo $row["content"];
    } else {
        echo 'Wrong password';  
    }

}

?>   

【问题讨论】:

  • 你想在 PHP 级别做吗?你可以很容易地用 htaccess 做到这一点(如果使用 apache)。 davidwalsh.name/password-protect-directory-using-htaccess 我不会在该站点上创建帐户以查看代码是什么,如果它使用 mysql_* 功能,则不安全。如果将用户数据直接传递给查询,则不安全。不过,有很多方法可以拒绝访问。
  • 所以您希望我们下载该软件包以查看其中的内容并运行它?我不会那样做;将您的代码粘贴到问题中。
  • 嗨,代码存储在数据库中的 id、html 内容和密码,我看到的问题,我是 php 和数据库的新手,脚本将密码存储为纯文本在数据库中
  • 怀疑它正在使用mysql_* 所以不,它不安全。如果使用 PHP 7,它甚至不会运行。
  • 谢谢,您知道一些或多或少正确的类似脚本吗?

标签: php passwords protected


【解决方案1】:

正如我在 cmets 中所说,您不应该再花时间在下载的内容上,因为它已经过时且不安全。

您可能以纯文本形式保存密码,这绝对不是一个好主意。

  • 是时候迈入 21 世纪了。

mysql_ API 已弃用,并已从 PHP 7.0 中完全删除。

您最好使用准备好的语句和password_hash() 或compatibility pack。

以下是一些参考资料:

注意使用mysql_real_escape_string() 并不能完全保证防止可能的 SQL 注入。

请参阅以下有关该主题的问答:

这是从一个或ircmaxell's 答案中提取的一段代码,它使用(PDO)准备好的语句和password_hash()。

来自:https://stackoverflow.com/a/29778421/1415724

只需使用库。严重地。它们的存在是有原因的。

不要自己做。如果您正在创建自己的盐,您做错了。您应该使用一个为您处理的库。

$dbh = new PDO(...);

$username = $_POST["username"];
$email = $_POST["email"];
$password = $_POST["password"];
$hash = password_hash($password, PASSWORD_DEFAULT);

$stmt = $dbh->prepare("insert into users set username=?, email=?, password=?");
$stmt->execute([$username, $email, $hash]);

在登录时:

$sql = "SELECT * FROM users WHERE username = ?";
$stmt = $dbh->prepare($sql);
$result = $stmt->execute([$_POST['username']]);
$users = $result->fetchAll();
if (isset($users[0]) {
    if (password_verify($_POST['password'], $users[0]->password) {
        // valid login
    } else {
        // invalid password
    }
} else {
    // invalid username
}

【讨论】:

  • 为什么这被否决了?我不明白这里的重点。我也不会删除它。直到你投反对票才能解释你为什么这样做。
  • 显然反对者​​不想说任何话,需要用评论来支持它,因为答案确实“回答”了问题。如果您不打算打扰,请发布您自己的答案。
  • 在这里,我会假装是反对者:我嫉妒你耳朵之间的灰质。为什么我一出生就带着我的无能? .好的,假装完毕:i.imgur.com/qF8BKHm.jpg
  • @Drew TBH,我希望有一个错误的答案/错误的语法,但在这种情况下没有;它回答了这个问题。如果他们这样做是因为我在某些情况下从 ircmaxell 的答案中提取了一些代码,那么至少可以这么说。我认为就我而言这很好,并且不能投票结束这个问题作为一个骗局。他们可能在靴子里发抖,因为他们认为我会说他们坏话什么的;我想他们知道我能做什么,但那不是重点;我并不难“交谈”和讨论它。他们想无知;很好。
猜你喜欢
  • 2013-03-22
  • 1970-01-01
  • 2017-10-27
  • 2015-06-25
  • 2020-01-19
  • 1970-01-01
  • 2012-11-04
  • 1970-01-01
  • 2011-10-09
相关资源
最近更新 更多