[Erledigt] Login Script lï¿½uft nicht auf dem Server

04.11.2005, 20:08

Hallo,

mein Login Script lï¿½uft bei mir lokal ohne Probleme aber auf dem Server funktioniert es einfach nicht. Nach dem Select wird das if else nicht ausgefï¿½hrt. Habe es schon mit mysql_num_rows und mysql_fetch_row probiert, hat aber nichts geholfen. Help!

PHP-Code:

  if(isset($login))

{

                            include "connect.inc.php";

                            $cpass=md5($pass);

 
                            $sqlab = "select `username`, `password` from `user` where `username`= '$user'";

                            $sqlab .= "and `password`= '$cpass'";

 
 
                            $_POST["res"] = mysql_query($sqlab)or die ("Error - Query failed: ".mysql_error());

 
 
 
                            $_POST["erg"]=mysql_fetch_object($_POST["res"]);

 
                            if($_POST["erg"]==FALSE)

                            {

 
                                echo "<div id=loginerror>Wrong username or password</div>";

 
                            }

                            else

                            {

                                echo "<div id=loginausgabe>Welcome [b]$user[/b].
";

                                echo"<input name='enter' class='button' type='button' value='>>Enter<<' onclick=location='setlink.php'></div>";

                                session_register("cpass");

 
                            }

 
 
    }

Zergling-new · 04.11.2005, 20:19

Hallo,

dein Code ist sehr schlecht, $_POST, $_GET, etc sollten unbedingt readonly sein, also bitte nur auslesen. Dass if-else nicht ausgeführt wird (weder if noch else) kann nicht sein.

http://tut.php-q.net

04.11.2005, 20:25

Stimmt schon $_Post macht hier wenig Sinn aber daran liegt es nicht, hab wieder auf readonly umgestellt. Irgendwie bekomme ich keinen Rückgabewert von mysql_fetch_object.
Tx für die schnelle Antwort

04.11.2005, 20:27

so ist es etwas sauberer:

PHP-Code:

  <?php

 
if (isset($login)) {

 
    // Wenn keine Verbindung zur Datenbank da ist,

    // kannst du schlecht Queries absetzen...

    // deshalb besser require();

    require("connect.inc.php");

 
    // Werte, die du an mysql übergibst, sollten

    // escape't werden, sonst mögliche SQL-Injection (google)

    $query=  "SELECT username FROM user WHERE username='". mysql_real_escape_string($user) ."' ";

    $qiery.= "AND password='". md5($pass) ."'";

 
    // Am besten gibst du mysql_query noch ein Datenbank-

    // handle mit, was von mysql_connect() zurückgegeben wird,

    // ist sauberer...

    $res= mysql_query($query, $dbhandle);

 
    // Dann guckst du einfach, ob du mindestens einen Daten-

    // satz zurückbekommen hast.

    if (mysql_num_rows($res) >= 1) {

 
        // Login erfolgreich

 
    } else {

 
        // Fehlerhafter Login

 
    }

 
?>

sollte nebenbei auch noch funktionieren

Gruss

04.11.2005, 20:51

Hi,
so hab jetzt meinen code mal gesäubert wie oben beschrieben. Sieht schon besser aus

. Habe gerade festgestellt das mit dem Select selber was nicht stimmt. Habt ihr ne Ahnung was es sein könnte? Auf dem lokalen Server läuft es ja??

04.11.2005, 22:08

Evtl. liegt es an der verschlüsselung, das select funktioniert bis username wenn ich noch zusätzlich password ausgeben möchte gehts nicht mehr

04.11.2005, 23:09

Hi,
so ich hab den Fehler, das Problem war, das die Spalte Password auf varchar(15) eingestellt war. Wird aber mit md5 verschlüsselt können max 255 Zeichen entstehen also hab ich die Spalte auf varchar(255) umgestellt und es läuft. Juhuii

04.11.2005, 23:11

Wer hat eigentlich das Gerücht in die Welt gesetzt, dass md5() eine Verschlüsselung ist?

robo47 · 05.11.2005, 10:12

Zitat:

Zitat von Tecnomaniac

Hi,
so ich hab den Fehler, das Problem war, das die Spalte Password auf varchar(15) eingestellt war. Wird aber mit md5 verschlüsselt können max 255 Zeichen entstehen also hab ich die Spalte auf varchar(255) umgestellt und es läuft. Juhuii

255 zeichen ? bei md5 ... dachte der hashwert hätte immer eine feste länge von 32 zeichen

-> http://de.wikipedia.org/wiki/Md5

imported_Ben · 05.11.2005, 12:12

Vielleicht, um noch ein paar Anregungen zu holen:

http://forum.developers-guide.net/thread54.html

Grüße Ben.

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
Script lastet Server zu hoch aus	echolot	PHP Tipps 2008	5	01.07.2008 10:58
script läuft nicht mehr seit server mit save mode on läuft	wolfram	PHP Tipps 2008	4	30.03.2008 16:29
script läuft nicht mehr seit server mit save mode on läuft	wolfram	PHP Tipps 2008	13	29.03.2008 21:10
Suche Login Script		PHP Tipps 2006	6	02.03.2006 11:09
Script funzt nicht.. server kanns nicht?		PHP Tipps 2007	12	23.11.2005 13:27
Problem mit Login Script		PHP Tipps 2007	4	15.11.2005 17:29
Login Script : Addon???	Chrisber	PHP Tipps 2005-2	0	27.09.2005 16:11
script läuft nur auf strato server !!!!!	janni	PHP Tipps 2005-2	7	02.07.2005 23:27
login mit cookies - Server schuld?		PHP Tipps 2005	19	03.06.2005 01:12
fehler im script oder fehler beim server		PHP Tipps 2005	12	21.05.2005 17:04
Login Script: sicher oder nicht sicher?		PHP Tipps 2005	26	23.02.2005 12:18
Gästebuch server Script trägt bei Gästebüchern keine einträg	Dueperthal	PHP Tipps 2004-2	12	28.11.2004 20:02
Login Script	Tschuu	PHP Tipps 2004-2	5	12.11.2004 22:33
Kleines Prob mit Login Script		PHP Tipps 2004	2	09.07.2004 01:39
Fehler im Login Script		PHP Tipps 2004	3	06.07.2004 20:37

04.11.2005, 20:08
Gast Beiträge: n/a	[Erledigt] Login Script lï¿½uft nicht auf dem Server Hallo, mein Login Script lï¿½uft bei mir lokal ohne Probleme aber auf dem Server funktioniert es einfach nicht. Nach dem Select wird das if else nicht ausgefï¿½hrt. Habe es schon mit mysql_num_rows und mysql_fetch_row probiert, hat aber nichts geholfen. Help! PHP-Code: if(isset($login)) { include "connect.inc.php"; $cpass=md5($pass); $sqlab = "select `username`, `password` from `user` where `username`= '$user'"; $sqlab .= "and `password`= '$cpass'"; $_POST["res"] = mysql_query($sqlab)or die ("Error - Query failed: ".mysql_error()); $_POST["erg"]=mysql_fetch_object($_POST["res"]); if($_POST["erg"]==FALSE) { echo "<div id=loginerror>Wrong username or password</div>"; } else { echo "<div id=loginausgabe>Welcome [b]$user[/b]. "; echo"<input name='enter' class='button' type='button' value='>>Enter<<' onclick=location='setlink.php'></div>"; session_register("cpass"); } }


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

04.11.2005, 20:19
Zergling-new Erfahrener Benutzer Registriert seit: 21.05.2008 Beiträge: 9.937	Hallo, dein Code ist sehr schlecht, $_POST, $_GET, etc sollten unbedingt readonly sein, also bitte nur auslesen. Dass if-else nicht ausgeführt wird (weder if noch else) kann nicht sein. http://tut.php-q.net

04.11.2005, 20:25
Gast Beiträge: n/a	Stimmt schon $_Post macht hier wenig Sinn aber daran liegt es nicht, hab wieder auf readonly umgestellt. Irgendwie bekomme ich keinen Rückgabewert von mysql_fetch_object. Tx für die schnelle Antwort

04.11.2005, 20:27
Gast Beiträge: n/a	so ist es etwas sauberer: PHP-Code: <?php if (isset($login)) { // Wenn keine Verbindung zur Datenbank da ist, // kannst du schlecht Queries absetzen... // deshalb besser require(); require("connect.inc.php"); // Werte, die du an mysql übergibst, sollten // escape't werden, sonst mögliche SQL-Injection (google) $query= "SELECT username FROM user WHERE username='". mysql_real_escape_string($user) ."' "; $qiery.= "AND password='". md5($pass) ."'"; // Am besten gibst du mysql_query noch ein Datenbank- // handle mit, was von mysql_connect() zurückgegeben wird, // ist sauberer... $res= mysql_query($query, $dbhandle); // Dann guckst du einfach, ob du mindestens einen Daten- // satz zurückbekommen hast. if (mysql_num_rows($res) >= 1) { // Login erfolgreich } else { // Fehlerhafter Login } ?> sollte nebenbei auch noch funktionieren Gruss

04.11.2005, 20:51
Gast Beiträge: n/a	Hi, so hab jetzt meinen code mal gesäubert wie oben beschrieben. Sieht schon besser aus . Habe gerade festgestellt das mit dem Select selber was nicht stimmt. Habt ihr ne Ahnung was es sein könnte? Auf dem lokalen Server läuft es ja??

04.11.2005, 22:08
Gast Beiträge: n/a	Evtl. liegt es an der verschlüsselung, das select funktioniert bis username wenn ich noch zusätzlich password ausgeben möchte gehts nicht mehr

04.11.2005, 23:09
Gast Beiträge: n/a	Hi, so ich hab den Fehler, das Problem war, das die Spalte Password auf varchar(15) eingestellt war. Wird aber mit md5 verschlüsselt können max 255 Zeichen entstehen also hab ich die Spalte auf varchar(255) umgestellt und es läuft. Juhuii

04.11.2005, 23:11
Gast Beiträge: n/a	Wer hat eigentlich das Gerücht in die Welt gesetzt, dass md5() eine Verschlüsselung ist?

05.11.2005, 12:12
imported_Ben Erfahrener Benutzer Registriert seit: 18.09.2003 Beiträge: 13.598 PHP-Kenntnisse: Fortgeschritten	Vielleicht, um noch ein paar Anregungen zu holen: http://forum.developers-guide.net/thread54.html Grüße Ben. __________________ Webentwickler-Blog - Buchbesprechung PHP Design Patterns - SQL-Antipatterns