ייעול קוד | טופס התחברות

[Jonathan Zeierman]

היי חברים,
אשמח להמלצות בנוגע לאבטחה / ייעול של הקוד אימות התחברות שלי.

PHP קוד:

<?php
###################################################################
##                     Strating Sessions.                             ##
###################################################################
session_start();
###################################################################
##                Set Cases For Functions.                         ##
###################################################################
switch ($_GET['page'])
{
    case 'login':
        login();
        break;
        
    default:
        login();
        break;
}
###################################################
    $security_code = rand(000000,999999);         ##
    $_SESSION['security_code'] = $security_code; ##
###################################################################
##                    Strating Login Function.                     ##
###################################################################
function Login()
{
    ################################################################################################
    ##                             Require A DB Connection File.                                      ##
    ################################################################################################
    require_once ('db/connect.php');                                                              ##
    ################################################################################################
   if (isset($_POST['login_check']))
   {
        ################################################################################################
        ##                                        Variables.                                              ##
        ################################################################################################
        $user = mysql_real_escape_string(htmlspecialchars($_POST['user']));                           ##
        $pass = mysql_real_escape_string(htmlspecialchars(md5($_POST['pass'])));                       ##
        $query = ("SELECT password, groups FROM users WHERE user = '".$user."'");                      ##
        $result = mysql_query($query)    or die("Mysql Getting The Next Error:" . mysql_error());      ##
        ################################################################################################
        if ($row = mysql_fetch_array($result))
        {
            if($row['password'] == $pass && $_SESSION['security_code'] == $_POST['security_code_check'] && $row['groups'] == 1)
            {
                $_SESSION['user'] = $user;
                $_SESSION['pass'] = $pass;
                $_SESSION['groups'] = $_SESSION['groups'] + 1;
                header('Location: index.php?page=Administrators');
            }
            if($row['password'] == $pass && $_SESSION['security_code'] == $_POST['security_code_check'] && $row['groups'] == 2)
            {
                $_SESSION['user'] = $user;
                $_SESSION['pass'] = $pass;
                $_SESSION['groups'] = $_SESSION['groups'] + 2;
                header('Location: index.php?page=Administrators_Deputys');
            }
            if($row['password'] == $pass && $_SESSION['security_code'] == $_POST['security_code_check'] && $row['groups'] == 3)
            {
                $_SESSION['user'] = $user;
                $_SESSION['pass'] = $pass;
                $_SESSION['groups'] = $_SESSION['groups'] + 3;
                header('Location: index.php?page=Workers');
            }
                mysql_close($db_connection); //Closing The Db Connection.
        }
        if(htmlspecialchars($_POST['user']) =='' || htmlspecialchars($_POST['pass'])=='' || htmlspecialchars($_POST['security_code_check'])=='')
        {
        echo 'אנא מלא את כל השדות';
        }
        else
        {
        echo 'שם משתמש ואו סיסמא שגויים';
        }
    }
}
?>

תודה לעוזרים, יונתן.

[אבי בר]

1)הכי חשוב לא ככה רושמים הערות.. זה רק עושה יותר נזק מתועלת, אתה יכול להסתכל על פרויקטים קוד פתוח ולראות איך נהוג:
www.github.com

בנוסף אין צורך לרשום הערות לדברים שהם ברורים מראש לדוגמה זה מיותר:

PHP קוד:

//if x is true
if (x==true)
//Strating Sessions.  
session_start(); 


2)המשפט switch שלך הוא מיותר כי לא משנה מה אתה תמיד קורא לפונקציה login() עדיף פשוט לקרוא לה.

3)הבדיקה שלך של הgroups לא הכי יעילה, בדרך כלל אם אתה רושם אותו קוד כמה פעמים זה אמור להדליק לך נורה אדומה.
תצמצם את הקוד כמה שיותר שלא תיהיה לך חזרה על אותו קוד יותר מפעם אחת

4) כבר לא משתמשים בספרייה mysql היא deprecated תעבור לmysqli או לpdo
http://php.net/manual/en/book.mysqli.php http://php.net/manual/en/ref.pdo-mysql.php

זה הערות כלליות, אם אתה רוצה לעלות טיפה ברמה כדאי שתתחיל לקרוא דברים, תקרא על oop.
תקרא על פונקציות נורמליות להצפנה ולא md5 שזה פריץ דוגמה: http://php.net/manual/en/function.password-hash.php

הsecurity_code לא ברור מה המטרה אבל הוא לא עוזר יותר מדי, אתה מנסה למנוע csrf? יש על זה גם המון מידע באינטרנט רק תקרא

סה"כ בהצלחה כולנו התחלנו ככה אבל זה סגנון קוד שמתאים לתחילת שנות ה2000 ולא בתקופתנו