Er det et sikkert log ind system?

Tags:    php

Dette logind system er skrevet i php med mysqli. Men vil gerne have at vide at det er sikkert. Og hvordan kan jeg bruge denne kode i en function :

index.php
Fold kodeboks ind/udPHP kode 


Så den fx kunne bruges sådan

Fold kodeboks ind/udPHP kode 


EDIT :

Fulde kode

Fold kodeboks ind/udPHP kode 


Er det et sikker login ellers må i godt komme med nogen eksempler.



Indlæg senest redigeret d. 18.11.2013 18:18 af Bruger #17607
7 svar postet i denne tråd vises herunder
0 indlæg har modtaget i alt 0 karma
Sorter efter stemmer Sorter efter dato
Er ikke sikkert nej.

Når du bruger prepared statements giver det ikke nogen mening at indsætte et user input direkte i din SQL query, du skal køre dem igennem din prepare og binde dem som variabler.

Fold kodeboks ind/udPHP kode 

Taget fra http://php.net/manual/en/mysqli.prepare.php



Grunden til jeg bruger user input er lidt for at sikre at folk ikke direkte ændre i databasen med ens kodeord så det ikke virker :).



Grunden til jeg bruger user input er lidt for at sikre at folk ikke direkte ændre i databasen med ens kodeord så det ikke virker :).


Det er du nødt til lige, at forklare igen



Grunden til jeg bruger user input er lidt for at sikre at folk ikke direkte ændre i databasen med ens kodeord så det ikke virker :).


Det er du nødt til lige, at forklare igen


Sagt på en anden måde jeg har min ps kode her :

Fold kodeboks ind/udPHP kode 


Det er det jeg mener med user input, og når folk skal skifte ens brugers kodeord fra en database bruger de nok en md5 generator og så sætter de deres nye kode ind, men de skal så også putte deres brugernavn i og salt :). Det synes jeg selv er en lille ekstra smart detajle for at lukke et lille hul :D.



ja en hash behøver du ikke at binde, men nu har du stadig nogle inputs direkte fra browseren



Du misforstår alt dette.. Jeg vil referere til http://mattbango.com/notebook/code/prepared-statements-in-php-and-mysqli/



prøv at skimme denne guide
http://www.eksperten.dk/guide/1480




t