Eerste script gemaakt! iemand tips waar ik op moet letten?

[Verwijderd]

Ik ben helemaal heppie heb me eerste script ooit gemaakt

Kan iemand hm bekijken en het script beoordelen? en heeft iemand tips waar ik op MOET letten?

Script:

code:

<?php
$database_gegevens = array("host"=>"localhost",
                           "username"=>"root",
                           "password"=>"Krijgjenietxx", 
                           "db"=>"database_x");
$errormsg          = array("mysqlcc" => "Kan niet verbinden met de Mysql Database.", 
                           "mysqldb" => "Er kon geen database met deze naam gevonden worden.", 
                           "logerror" => ""); 
function dbvervinden() 
{
global $database_gegevens; global $errormsg;
$mysqlcon = mysql_connect($database_gegevens[host], $database_gegevens[username], $database_gegevens[password]);
if ($mysqlcon) {
        $selectdb = mysql_select_db($database_gegevens[db]);
        if ($selectdb) {
                session_start();
                    if(!session_is_registered(ses10n)){
                        echo '';
                } else mysql_close($mysqlcon);
            $datum = $_SERVER['REQUEST_TIME']; $ip = $_SERVER['REMOTE_ADDR']; $useragent = $_SERVER['HTTP_USER_AGENT'];
            if (mysql_query("INSERT INTO `logs` (`datum`, `ip`, `useragent`) VALUES ('".$datum."', '".$ip."', '".$useragent."')")){
                } else echo($errormsg['logerror']); mysql_close($mysqlcon);
            } else echo $errormsg['mysqldb']; mysql_close($mysqlcon);
    } else echo $errormsg['mysqlcc']; mysql_close($mysqlcon); 
}


?>

Grt, Romano

[Osiris]

Iets over je gebruikte MySQL-API en iets over iets wat gruwelijk fout bij je kan gaan.

[Fish]

nooit je root account gebruiken, aparte gebruiker met een minimale set rechten aanmaken

ook een user agent of een url kan sql bevatten dus altijd escapen

[Ahrnuld]

Idd gebruik de mysqli functies ipv mysql (let op de i). De mysql functies die je nu gebruikt zijn verouderd en worden bij nieuwere versies van PHP niet meer ondersteund.

Maar vertel, wat is het doel van je script? Gebruik je de functie 'dbvervinden'(?!) nog ergens?

[Verwijderd]

BerendBoon schreef op woensdag 25 december 2013 @ 00:28:
Idd gebruik de mysqli functies ipv mysql (let op de i). De mysql functies die je nu gebruikt zijn verouderd en worden bij nieuwere versies van PHP niet meer ondersteund.

Maar vertel, wat is het doel van je script? Gebruik je de functie 'dbvervinden'(?!) nog ergens?

Ik probeer het zo veilig mogelijk te maken maar ik zie de reactie van fish en het is dus niet zo veilig dan ik dacht de functie van dit script is gewoon verbinden naar mijn database

[Verwijderd]

fish schreef op woensdag 25 december 2013 @ 00:27:
nooit je root account gebruiken, aparte gebruiker met een minimale set rechten aanmaken

ook een user agent of een url kan sql bevatten dus altijd escapen

Ah bedankt voor deze tips! ik ga er gelijk wat mee doen

[Verwijderd]

Osiris schreef op woensdag 25 december 2013 @ 00:27:
Iets over je gebruikte MySQL-API en iets over iets wat gruwelijk fout bij je kan gaan.

bedankt voor deze twee links deze ga ik zeker even door nemen

[Osiris]

BerendBoon schreef op woensdag 25 december 2013 @ 00:28:
Maar vertel, wat is het doel van je script? Gebruik je de functie 'dbvervinden'(?!) nog ergens?

Hij zal wel `dbverbinden` bedoelen.. Maar zolang 'ie die niet aanroept, is 't script wel ultiem veilig

Ennuh, dude, 3x achter elkaar reageren is beetje onzinnig, edit asjeblieft je vorige reactie i.p.v. een nieuwe te plaatsen.

[ Voor 16% gewijzigd door Osiris op 25-12-2013 00:32 ]

[danslo]

• Globals die nergens voor nodig zijn.
• Gebruik mysqli of PDO i.p.v. mysql.
• Waardeloze indentatie
• session_is_registered is deprecated
• Gebruik quotes als je een string meegeeft aan een functie

... en zo kunnen we nog wel een tijdje door gaan. Wat wil je precies bereiken?

[RobIII]

...en verder doen we niet aan code-reviews hier (dat valt onder sitechecks).
Probeer 't eens bij http://codereview.stackexchange.com/ ofzo?

Verder: Gebruik a.u.b. de wijzig-link (rechtsbovenaan je post) als je iets toe te voegen hebt; je topic herhaaldelijk omhoogschoppen is niet nodig en die melding staat er niet voor niets:

[ Voor 63% gewijzigd door RobIII op 25-12-2013 01:00 ]