בניתי מיני מערכת להתחברות מנהלים לאתר שלי. יש סך הכול מנהל אחד שהוא אני...
עשיתי התחברות בצורה המוצגת להלן.. האם התחברות כזו בטוחה ? לפי מה שאני יודע זו ההתחברות הבטוחה ביותר שקיימת, כיוון שהיא לא מתעסקת עם בסיס נתונים...

הפונקצייה printlogin מדפיסה קוד HTML של טופס ההתחברות. טופס ההתחברות טוען את הקוד הנל ושולח משתנה mod? במידה ומשתנה זה מכיל את המחרוזת login המערכת מנסה לחבר את המנהל אל לוח הבקרה

קוד PHP:

 $user = $_POST['user'];
$pass = $_POST['pass'];
$mod = $_POST['mod'];

if($mod=="login")
{
        if($user=="admin" && $pass=="123456")
        {
              echo "פאנל ניהול";
        }
        else
        {
              printlogin();
        }
}
else
{
        printlogin();
} 


ניתן לראות את פאנל המנהלים כאן עם שם המשתמש והסיסמא שאתם רואים כאן בקוד
http://uzone-il.net/admin.php

כן זה בטוח...

רק שאולי כדאי גם שתיצור HASH של השם משתמש וסיסמא, ותשווה את הקלט להאש.
לדוגמא:

קוד PHP:

 if (md5($_POST['password']) == '[md5Hash]')
 ... 


הערת אגב - הקוד הבא נראה מיותר:

קוד PHP:

 $user = $_POST['user'];
$pass = $_POST['pass'];
$mod = $_POST['mod']; 


חשוב מאוד ליצור HASH לסיסמא!
אתה שם את הקוד שלך על שרת שאני מניח שהוא לא שלך והוא לא DEDICATED
לכן, האדמיניסטרטור של השרת יכול לראות את כל הקבצים על השרת שלו, כולל הקובץ הזה (בוא נניח שהשרת מאובטח. אבל קיימת גם אפשרות שהוא פרוץ ויש הרבה אנשים עם גישת ROOT).

לכן אם הוא מסתכל על הקובץ PHP, הוא יראה את הסיסמא הלא מוצפנת ויוכל להשתמש בה

תשכח עכשיו מהעובדה שאם הוא ROOT אז הוא יכול לשנות את הקובץ (והסיסמא שלך). אנחנו מדברים על תיאוריה בה מישהו מסוים קיבל הרשאות קריאה בלבד לקובץ הזה, ובתיאוריה אם יש לך plaintext אז יותר קל לתקוף את המערכת..

שירשרת לא נכון....
אתה אומר לי בעצם במקום לכתוב בקובץ 123456, לכתוב את ההצפנה בMD5 של 123456
ולהפוך את הסיסמא שהתקבלה לMD5 ולהשוות את הסיסמאות ?

למה שרשרתי לא נכון?
אני הדגשתי את מה שDCD אמר

תכתוב סקריפט קטן שנותן HASH של מחרוזת ואז אחרי שתבחר סיסמא, שים את הHASH שמתקבל ממנה במשוואה שלך.

שזה בעצם מה שאתה אמרת

אחלה. כל הכבוד!

יותר נוח לי ככה להפוך את המשתני POST למשתנים פשוטים.... אני יודע שזה מיותר,
סך הכול.זה לא תופס יותר מדיי משאבים

עריכה: התכוונתי להגיב לDCD

תיאורתית לא רצוי להוסיף את הפונקציה mysql_escape_string
לנתונים שאתה מקבל מהמשתמש?

זה לא רצוי בכלל.

אבל רצוי, גם תאורטית וגם מעשית, להוסיף את הפונקציה mysql_real_escape_string לכל נתון חיצוני שאתה מקבל.