php-resource



Zurück   PHP-Scripte PHP-Tutorials PHP-Jobs und vieles mehr > Entwicklung > PHP Developer Forum
 

Login

 
eingeloggt bleiben
star Jetzt registrieren   star Passwort vergessen
 

 

 


PHP Developer Forum Hier habt ihr die Möglichkeit, eure Skriptprobleme mit anderen Anwendern zu diskutieren. Seid so fair und beantwortet auch Fragen von anderen Anwendern. Dieses Forum ist sowohl für ANFÄNGER als auch für PHP-Profis! Fragen zu Laravel, YII oder anderen PHP-Frameworks.

Antwort
 
LinkBack Themen-Optionen Thema bewerten
  #1 (permalink)  
Alt 17-03-2015, 20:06
ShadowSniper
 Registrierter Benutzer
Links : Onlinestatus : ShadowSniper ist offline
Registriert seit: Mar 2015
Beiträge: 7
ShadowSniper befindet sich auf einem aufstrebenden Ast
Question sichere Benutzerauthentifizierung?

Hallo zusammen,

ich bin neu hier und habe noch recht wenig PHP-Kentnisse.
Ich versuche zur Zeit eine sichere(!) Benutzerauthentifizierung zu implementieren.
Momentan bin ich noch beim Script, um einen neuen User in die Datenbank einzutragen. Dabei gehe ich folgendermaßen vor:

- DB-Access über Include einbinden
- DB-Verbindung aufbauen
- Benutzereingaben mit FILTER_SANITIZE_STRING filtern
- Passwort mit password_hash() und mcrypt_create_iv() verschlüsseln
- DB-Eintrag mit prepared statements durchführen
- DB-Verbindung schließen

Beim ausführen erhalte ich folgende Fehlermeldung:
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ':user, assword, :email, :ip)' at line 2

Könnt ihr mir sagen wo der Fehler liegt? Außerdem würde gerne wissen wie ich das Passwort noch besser verschlüsseln kann und das Script im Allgemeinen sicherer machen kann.

Hier ist mein kompletter Quellcode:

PHP-Code:
<?php
header
('Content-Type: text/html; charset=utf-8');
include_once 
__DIR__ '/includes/access/access.php';

$mysql mysqli_connect($db_host$db_user$db_pwd$db_name);

if (
mysqli_connect_errno()) {
 echo 
"<p>Fehler: Verbindung zur Datenbank nicht möglich. Versuchen Sie es zu einem späteren Zeitpunkt nochmal.</p>";
 exit();
}

$user filter_input(INPUT_POST'user'FILTER_SANITIZE_STRING);
$password filter_input(INPUT_POST'password'FILTER_SANITIZE_STRING);
$email NULL;
$ip NULL;

if (!
user || !$password) {
echo 
"<p>Bitte füllen Sie alle Felder aus.</p>";
exit();
}

$options = [
    
'cost' => 11,
    
'salt' => mcrypt_create_iv(22MCRYPT_DEV_URANDOM),
];

$password password_hash($passwordPASSWORD_BCRYPT$options);

$query "INSERT INTO users (user, password, email, ip) VALUES
        (:user, :password, :email, :ip)"
;

if (
$stmt mysqli_prepare ($mysql$query)) {
    
mysqli_stmt_bind_Param($stmt':user'$user);
    
mysqli_stmt_bind_Param($stmt':password'$password);
    
mysqli_stmt_bind_Param($stmt':email'$email);
    
mysqli_stmt_bind_Param($stmt':ip'$ip);
    
    
mysqli_stmt_execute($stmt);

    echo 
"Der Benutzer wurde erfolgreich hinzugefügt.";
}

else {
    echo 
$mysql -> error;
}

mysqli_close($mysql);
?>
Mit Zitat antworten
  #2 (permalink)  
Alt 17-03-2015, 20:30
h3ll
 Registrierter Benutzer
Links : Onlinestatus : h3ll ist offline
Registriert seit: Mar 2008
Beiträge: 3.578
h3ll befindet sich auf einem aufstrebenden Ast
Standard

Wie kommst du auf die Idee, dass man bei mysqli_prepare() :user, :password, usw. als Platzhalter verwenden kann?
Mit Zitat antworten
  #3 (permalink)  
Alt 17-03-2015, 20:36
ShadowSniper
 Registrierter Benutzer
Links : Onlinestatus : ShadowSniper ist offline
Registriert seit: Mar 2015
Beiträge: 7
ShadowSniper befindet sich auf einem aufstrebenden Ast
Standard

Von dieser Seite: PHP: Prepared Statements und Stored Procedures - Manual
Aber anscheinend habe ich da was falsch verstanden!
Mit Zitat antworten
  #4 (permalink)  
Alt 17-03-2015, 20:38
h3ll
 Registrierter Benutzer
Links : Onlinestatus : h3ll ist offline
Registriert seit: Mar 2008
Beiträge: 3.578
h3ll befindet sich auf einem aufstrebenden Ast
Standard

Zitat:
Zitat von ShadowSniper Beitrag anzeigen
Von dieser Seite: PHP: Prepared Statements und Stored Procedures - Manual
Aber anscheinend habe ich da was falsch verstanden!
Das ist die Doku zu PDO und nicht zu mysqli. Wobei ich dir empfehlen würde gleich auf PDO umzusteigen.
Mit Zitat antworten
  #5 (permalink)  
Alt 17-03-2015, 20:42
ShadowSniper
 Registrierter Benutzer
Links : Onlinestatus : ShadowSniper ist offline
Registriert seit: Mar 2015
Beiträge: 7
ShadowSniper befindet sich auf einem aufstrebenden Ast
Standard

Wie müsste das denn aussehen mit PDO??
Habe ich schon probiert, hat aber auch nicht so funktioniert wie ich das wollte.
Mit Zitat antworten
  #6 (permalink)  
Alt 17-03-2015, 20:48
h3ll
 Registrierter Benutzer
Links : Onlinestatus : h3ll ist offline
Registriert seit: Mar 2008
Beiträge: 3.578
h3ll befindet sich auf einem aufstrebenden Ast
Standard

Zitat:
Zitat von ShadowSniper Beitrag anzeigen
Wie müsste das denn aussehen mit PDO??
So wie es in der Doku zu PDO steht
Mit Zitat antworten
  #7 (permalink)  
Alt 18-03-2015, 00:27
rossixx
 Registrierter Benutzer
Links : Onlinestatus : rossixx ist offline
Registriert seit: Jul 2003
Ort: Berlin
Beiträge: 514
Blog-Einträge: 2
rossixx wird schon bald berühmt werden
Question pro und contra

PDO oder mysqli ???
Mit Zitat antworten
  #8 (permalink)  
Alt 18-03-2015, 12:46
ShadowSniper
 Registrierter Benutzer
Links : Onlinestatus : ShadowSniper ist offline
Registriert seit: Mar 2015
Beiträge: 7
ShadowSniper befindet sich auf einem aufstrebenden Ast
Standard

Ich möchte das ganze jetzt mit PDO machen.
Mein Problem ist das ich nicht richtig weis wie man damit die DB-Verbindung herstellt.
Ich habe danach gesucht und den Code jetzt so geändert:

PHP-Code:
<?php
header
('Content-Type: text/html; charset=utf-8');
include_once 
__DIR__ '/includes/access/access.php';

$user filter_input(INPUT_POST'user'FILTER_SANITIZE_STRING);
$password filter_input(INPUT_POST'password'FILTER_SANITIZE_STRING);
$email NULL;
$ip NULL;

if (!
user || !$password) {
echo 
"<p>Bitte füllen Sie alle Felder aus.</p>";
exit();
}

$options = [
    
'cost' => 11,
    
'salt' => mcrypt_create_iv(22MCRYPT_DEV_URANDOM),
];

$password password_hash($passwordPASSWORD_BCRYPT$options);

try {
    
$dbh = new PDO('mysql:host=$db_host;dbname=$db_name'$db_user$db_pwd);

    
$query "INSERT INTO users (user, password, email, ip) VALUES
            (:user, :password, :email, :ip)"
;

    
$stmt $dbh->prepare($query);

    
$stmt->bindParam(':user'$user);
    
$stmt->bindParam(':password'$password);
    
$stmt->bindParam(':email'$email);
    
$stmt->bindParam(':ip'$ip);
        
    
$stmt->execute();

    echo 
"Der Benutzer wurde erfolgreich hinzugefügt.";    
}

catch (
PDOException $e) {
   print 
"Error!: " $e->getMessage() . "<br/>";
   die();
}
?>
Diese Fehlermeldungen werden mir angezeigt:

Warning: PDO::__construct(): php_network_getaddresses: getaddrinfo failed: Name or service not known in ###/php/add_user.php on line 32
Error!: SQLSTATE[HY000] [2002] php_network_getaddresses: getaddrinfo failed: Name or service not known

Also muss irgendwas mit bindParam() nicht stimmen oder?
Mit Zitat antworten
  #9 (permalink)  
Alt 18-03-2015, 13:24
ShadowSniper
 Registrierter Benutzer
Links : Onlinestatus : ShadowSniper ist offline
Registriert seit: Mar 2015
Beiträge: 7
ShadowSniper befindet sich auf einem aufstrebenden Ast
Standard

Ich habe nun noch eine Server-Variable eingefügt und danach kommt nur noch diese Fehlermeldung:

Error!: SQLSTATE[HY000] [2002] Connection refused
Mit Zitat antworten
  #10 (permalink)  
Alt 18-03-2015, 16:10
ShadowSniper
 Registrierter Benutzer
Links : Onlinestatus : ShadowSniper ist offline
Registriert seit: Mar 2015
Beiträge: 7
ShadowSniper befindet sich auf einem aufstrebenden Ast
Standard

Ich habe erneut den Code verändert. Jetzt funktioniert das registrieren,
aber die Anmeldung leider noch nicht. Kann mir Jemand sagen was an der Abfrage mit password_verify() in login.php falsch ist?

register.php
PHP-Code:
<?php 
    
require_once __DIR__ '/common.php';

    
$username filter_input(INPUT_POST'username'FILTER_SANITIZE_STRING);
    
$userPassword filter_input(INPUT_POST'password'FILTER_SANITIZE_STRING); 
     
    if(!empty(
$_POST)) 
    { 
        if(empty(
$_POST['username'])) 
        { 
            die(
"Please enter a username."); 
        } 
         
        if(empty(
$_POST['password'])) 
        { 
            die(
"Please enter a password."); 
        } 

        if(!
filter_var($_POST['email'], FILTER_VALIDATE_EMAIL)) 
        { 
            die(
"Invalid E-Mail Address"); 
        } 
         
        
$query 
            SELECT 
                1 
            FROM users 
            WHERE 
                user = :username 
        "

         
        
$query_params = array( 
            
':username' => $username
        
); 
         
        try 
        { 
            
$stmt $db->prepare($query); 
            
$result $stmt->execute($query_params); 
        } 
        catch(
PDOException $ex
        { 
            die(
"Failed to run query: " $ex->getMessage()); 
        } 

        
$row $stmt->fetch(); 
         
        if(
$row
        { 
            die(
"This username is already in use"); 
        } 

        
$query 
            SELECT 
                1 
            FROM users 
            WHERE 
                email = :email 
        "

         
        
$query_params = array( 
            
':email' => $_POST['email'
        ); 
         
        try 
        { 
            
$stmt $db->prepare($query); 
            
$result $stmt->execute($query_params); 
        } 
        catch(
PDOException $ex
        { 
            die(
"Failed to run query: " $ex->getMessage()); 
        } 
         
        
$row $stmt->fetch(); 
         
        if(
$row
        { 
            die(
"This email address is already registered"); 
        } 
 
        
$query 
            INSERT INTO users ( 
                user, 
                password, 
                email 
            ) VALUES ( 
                :username, 
                :password, 
                :email 
            ) 
        "

         
        
$options = [
            
'cost' => 11,
            
'salt' => mcrypt_create_iv(22MCRYPT_DEV_URANDOM),
        ];

        
$password password_hash($passwordPASSWORD_BCRYPT$options);
         
        
$query_params = array( 
            
':username' => $username
            
':password' => $password
            
':email' => $_POST['email'
        ); 
         
        try 
        { 
            
$stmt $db->prepare($query); 
            
$result $stmt->execute($query_params); 
        } 
        catch(
PDOException $ex
        { 
            die(
"Failed to run query: " $ex->getMessage()); 
        } 

        
header("Location: login.php"); 
         
        die(
"Redirecting to login.php"); 
    } 
     
?> 
<h1>Register</h1> 
<form action="register.php" method="post"> 
    Username:<br /> 
    <input type="text" name="username" value="" /> 
    <br /><br /> 
    E-Mail:<br /> 
    <input type="text" name="email" value="" /> 
    <br /><br /> 
    Password:<br /> 
    <input type="password" name="password" value="" /> 
    <br /><br /> 
    <input type="submit" value="Register" /> 
</form>
login.php
PHP-Code:
<?php 
    
require_once __DIR__ '/common.php';

    
$username filter_input(INPUT_POST'username'FILTER_SANITIZE_STRING);
    
$userPassword filter_input(INPUT_POST'password'FILTER_SANITIZE_STRING); 
     
    
$submitted_username ''
     
    if(!empty(
$_POST)) 
    { 
        
$query 
            SELECT 
                user, 
                password, 
                email 
            FROM users 
            WHERE 
                user = :username 
        "

         
        
$query_params = array( 
            
':username' => $username 
        
); 
         
        try 
        { 
            
$stmt $db->prepare($query); 
            
$result $stmt->execute($query_params); 
        } 
        catch(
PDOException $ex
        { 
            die(
"Failed to run query: " $ex->getMessage()); 
        } 
         
        
$login_ok false
         
        
$row $stmt->fetch(); 
        if(
$row
        {                                  
            if(
password_verify($userPassword$row['password'])) 
            { 
                
$login_ok true
            } 
        } 
         
        if(
$login_ok
        { 
            unset(
$row['password']);
            
$_SESSION['user'] = $row;  
             echo 
"Hier ist die geheime Seite!";
        } 
        else 
        { 
            print(
"Login Failed.");       
            
$submitted_username htmlentities($usernameENT_QUOTES'UTF-8'); 
        } 
    }     
?> 
<h1>Login</h1> 
<form action="login.php" method="post"> 
    Username:<br /> 
    <input type="text" name="username" value="<?php echo $submitted_username?>" /> 
    <br /><br /> 
    Password:<br /> 
    <input type="password" name="password" value="" /> 
    <br /><br /> 
    <input type="submit" value="Login" /> 
</form>
Mit Zitat antworten
  #11 (permalink)  
Alt 18-03-2015, 18:37
ShadowSniper
 Registrierter Benutzer
Links : Onlinestatus : ShadowSniper ist offline
Registriert seit: Mar 2015
Beiträge: 7
ShadowSniper befindet sich auf einem aufstrebenden Ast
Standard

So wie ich das verstanden habe nimmt password_verify() den Hash,
der von der DB kommt und überprüft ob er mit dem vom Benutzer eingegebenen Passwort übereinstimmt. Richtig?

Ich habe zur Überprüfung den Hash und das Passwort in Klartext übergeben, aber auch das hat leider nicht funktioniert.
Mit Zitat antworten
Antwort

Lesezeichen

Stichworte
benutzerauthentifizierung, login, sicherheit


Aktive Benutzer in diesem Thema: 1 (Registrierte Benutzer: 0, Gäste: 1)
 

Ähnliche Themen
Thema Autor Forum Antworten Letzter Beitrag
[Script] Live Editing einer Website direkt im Browser mit Benutzerauthentifizierung technoid Apps und PHP Script Gesuche 2 01-05-2006 18:00
Sichere Abfragen? PHP-Desaster PHP Developer Forum 10 14-03-2006 18:02
sichere Sessionverwaltung?!? galaxy2 PHP Developer Forum 6 04-01-2006 15:55
Sichere Loginmethode hasch PHP Developer Forum 13 23-08-2005 23:20
sichere Dokumentenablage ? iceman BRAINSTORMING PHP/SQL/HTML/JS/CSS 6 20-08-2003 11:03

Themen-Optionen
Thema bewerten
Thema bewerten:

Forumregeln
Es ist Ihnen nicht erlaubt, neue Themen zu verfassen.
Es ist Ihnen nicht erlaubt, auf Beiträge zu antworten.
Es ist Ihnen nicht erlaubt, Anhänge hochzuladen.
Es ist Ihnen nicht erlaubt, Ihre Beiträge zu bearbeiten.

BB-Code ist an.
Smileys sind an.
[IMG] Code ist an.
HTML-Code ist aus.
Trackbacks are an
Pingbacks are an
Refbacks are an


PHP News

ebiz-trader 7.5.0 mit PHP7 Unterstützung veröffentlicht
ebiz-trader 7.5.0 mit PHP7 Unterstützung veröffentlichtDie bekannte Marktplatzsoftware ebiz-trader ist in der Version 7.5.0 veröffentlicht worden.

28.05.2018 | Berni

Wissensbestand in Unternehmen
Wissensbestand in UnternehmenLebenslanges Lernen und Weiterbilden sichert Wissensbestand in Unternehmen

25.05.2018 | Berni


 

Aktuelle PHP Scripte

PHP Server Monitor

PHP Server Monitor ist ein Skript, das prüft, ob Ihre Websites und Server betriebsbereit sind.

11.09.2018 Berni | Kategorie: PHP/ Security
PHP WEB STATISTIK ansehen PHP WEB STATISTIK

Die PHP Web Statistik bietet Ihnen ein einfach zu konfigurierendes Script zur Aufzeichnung und grafischen und textuellen Auswertung der Besuchern Ihrer Webseite. Folgende zeitlichen Module sind verfügbar: Jahr, Monat, Tag, Wochentag, Stunde Folgende son

28.08.2018 phpwebstat | Kategorie: PHP/ Counter
Affilinator - Affilinet XML Produktlisten Skript

Die Affilinator Affilinet XML Edition ist ein vollautomatisches Skript zum einlesen und darstellen der Affili.net (Partnerprogramm Netzwerk) Produktlisten und Produktdaten. Im Grunde gibt der Webmaster seine Affilinet PartnerID ein und hat dann unmittelb

27.08.2018 freefrank@ | Kategorie: PHP/ Partnerprogramme
 Alle PHP Scripte anzeigen

Alle Zeitangaben in WEZ +2. Es ist jetzt 04:59 Uhr.