Lijst ene klantnummer wel, andere niet.

T.Nijborg

Heey allemaal,

Ik ben een klantenpaneel aan het maken, en nou ben ik bezig met de login, nou heb ik 1 probleem ermee, mijn klantnummer leest die wel, maar van een 2, test account niet. Misschien zien jullie het probleem.

De code voor gegevens uit de db:

PHP

// STANDAARD GEGEVENS BEGIN //
$info = mysqli_query($con, "SELECT * FROM customers");
$gegevens = mysqli_fetch_array($info);

De code die checkt of de gegevens kloppen:

PHP

<?php
// SYSTEEM INLOGGEN BEGIN


// Klantnummer En wachtwoord
$klantnummer = $_REQUEST['klant'];
$wachtwoord = $_REQUEST['pass'];


//Form openen
if(isset($_POST['login'])){


//Pass en klant beveilligen
$klant = md5(sha1($klantnummer));
$pass = md5(sha1($wachtwoord));


// checks
if($gegevens['klantnummer'] == $klant){
if($gegevens['wachtwoord'] == $pass){


//Sessie zetten
$_SESSION['user'] = $klant;
header('Refresh: 5; url=index.php');


} else{
$fout = $lang_pass_fout;
}
} else {
$fout = $lang_klantnummer_fout;
}


//Form sluiten
}


// SYSTEEM INLOGGEN EINDE


if($_SESSION['user'] == ""){
?>

Meer tonen

Ik heb alles geprobeerd.. Iemand enig idee?

Gr,

Reza

Zou je jouw hele code kunnen posten op pastebin.com en de link hier delen?
Verder is jouw code, naar mijn mening, iets te omslachtig qua 'efficiëntie' en zou ik een inlog systeem anders aanpakken.

Qua debuggen zou ik nu even eens var_dump uitproberen, dus:

PHP

var_dump($gegevens['klantnummer'], $gegevens['wachtwoord'], $klant, $wachtwoord);

Zo kun je al kijken of de gegevens ook echt kloppen, verder snap ik niet echt waarom je het klantnummer beveiligd?

T.Nijborg

puur voor de veilligheid, en dit is de code die ervoor zorgt dat dit het doet, hele codes ga ik niet opzetten. Maar hij pakt maar 1 rij uit de database, en dat snap ik niet ...

bramieboy100

PHP

// STANDAARD GEGEVENS BEGIN //
$info = mysqli_query($con, "SELECT * FROM customers");
$gegevens = mysqli_fetch_array($info);

Hier selecteer je alle rijen en alle kolommen waardoor er een array in een array is. Hou je hier wel rekening mee? Ik zie dat namelijk nergens in je script staan.

Het is inderdaad onnodig dat je je klantnummer codeert omdat het handiger is als je je klantnummer in je klantenpaneel kan zien. Zo kan je je klantnummer meteen doorgeven en je hebt alle info. Verder als je iets codeert doe dit dan niet dubbel omdat de hash dan minder uniek wordt. Daarbij is ook het risico dat je twee verschillende klantnummers hebt maar met de dezelfde hash.

T.Nijborg

Hoe werkt het dan bramieboy100? want ik snap er niks meer van-,-. Verrote mysqli

bramieboy100

Je hebt tenminste een gegeven nodig om geen array in een array te krijgen dat uniek is. Bijvoorbeeld een id. Dan haal je dit op met een SQL query en dan kan je $gegevens['password'] bij wijze van spreken doen. Maar mij is nog niet helemaal duidelijk wat je precies wil bereiken.

Mag ik trouwens vragen waarom je request gebruikt en niet post. Request zorgt ervoor dat je get en post kan gebruiken.

T.Nijborg

Hallo,

Punt is dat ik niet veel snap van die mysqli... En wat ik probeer is een inlog systeem te bouwen..

Dit is de code die ik nu heb, maar dat pakt alleen de eerste rij waardoor je alleen met de gegevens vanaf de 1ste rij in de tabel kan inloggen, hoe selecteer ik alle rijen

PHP

// SYSTEEM INLOGGEN BEGIN


//database
$result = mysqli_query($con,"SELECT * FROM customers");
$gegevens = mysqli_fetch_array($result);


//Form openen
if(isset($_POST['login'])){


// Klantnummer En wachtwoord
$klant = $_POST['klant'];
$wachtwoord = $_POST['pass'];
//Pass en klant beveilligen
$pass = sha1($wachtwoord);
  
// checks
if($gegevens['klantnummer'] == $klant){
if($gegevens['wachtwoord'] == $pass){




//Sessie zetten
$_SESSION['user'] = $klant;
header('Refresh: 5; url=index.php');


} else{
$fout = $lang_pass_fout;
}
} else {
$fout = $lang_klantnummer_fout;
}


//Form sluiten
}




// SYSTEEM INLOGGEN EINDE

Meer tonen

Reza

" SELECT klantnummer,wachtwoord FROM customers WHERE klantnummer = '".$_POST['klant']." "

Dat stukje moet natuurlijk nu wel na de isset en na het stukje dat je $_POST omzet naar variabelen. Ook zou ik de input van de user beveiligen.'

T.Nijborg

Nu heb ik dit en nog steeds niet werkend...

PHP

// SYSTEEM INLOGGEN BEGIN


//Form openen
if(isset($_POST['login'])){


// Klantnummer En wachtwoord
$klantnummer = $_POST['klant'];
$wachtwoord = $_POST['pass'];
//Pass en klant beveilligen
$pass = sha1($wachtwoord);


//database
$result = mysqli_query($con,"SELECT klantnummer,wachtwoord FROM customers WHERE klantnummer = '".$_POST['klant']."'");
$gegevens = mysqli_fetch_array($result);
  
// checks
if($gegevens['klantnummer'] == $klantnummer){
if($gegevens['wachtwoord'] == $pass){




//Sessie zetten
$_SESSION['user'] = $klant;
header('Refresh: 5; url=index.php');


} else{
$fout = $lang_pass_fout;
}
} else {
$fout = $lang_klantnummer_fout;
}


//Form sluiten
}




// SYSTEEM INLOGGEN EINDE

Meer tonen

Reza

Nogmaals als het verkeerd resultaat geeft, moet je het gaan debuggen. Zet ergens var_dump() neer(met de variabelen erin) en kijk wat eruit komt. Zo weet je of de query wel het goede resultaat ophaalt of niet...

bramieboy100

Alle rijen selecteren is een erg omslachtige manier en ook onveilig. Selecteer alleen de data die je nodig hebt dus bijv SELECT * FROM tabel WHERE klantnummer=5012.

L.Kok

Het zit hem in de controle van je wachtwoord, de laatste if die je hebt staan voordat je de session gaat zetten.
Naar mijn mening moet je deze query gebruiken:

PHP

$result = mysqli_query($con,"SELECT klantnummer,wachtwoord FROM customers WHERE klantnummer = '".$_POST['klant']."' AND wachtwoord = '".sha1($_POST['pass'])."'");

Op voorwaarde dat het database veld voor wachtwoord, wachtwoord noemt.
Ik zou tevens ook eens kijken naar het beveiligen van je query.

Bovenstaande query kan je gewoon een count rows op los laten om te kijken of er een match is in de database.

T.Nijborg

Citaat van L.Kok
Het zit hem in de controle van je wachtwoord, de laatste if die je hebt staan voordat je de session gaat zetten.
Naar mijn mening moet je deze query gebruiken:
PHP
$result = mysqli_query($con,"SELECT klantnummer,wachtwoord FROM customers WHERE klantnummer = '".$_POST['klant']."' AND wachtwoord = '".sha1($_POST['pass'])."'");
Op voorwaarde dat het database veld voor wachtwoord, wachtwoord noemt.
Ik zou tevens ook eens kijken naar het beveiligen van je query.
Bovenstaande query kan je gewoon een count rows op los laten om te kijken of er een match is in de database.

Beveilliging doe ik zodra het werkt, ik heb nu dit:

PHP

// SYSTEEM INLOGGEN BEGIN


//Form openen
if(isset($_POST['login'])){


// Klantnummer En wachtwoord
$klantnummer = $_POST['klant'];
$wachtwoord = $_POST['pass'];
//Pass en klant beveilligen
$pass = sha1($wachtwoord);


//database
$gegevens = "SELECT klantnummer, wachtwoord FROM customers WHERE klantnummer='$klantnummer' 
AND wachtwoord='$pass'";




$result = mysqli_query($con,$gegevens)or die(mysqli_error());
$num_row = mysqli_num_rows($result);
$row=mysqli_fetch_array($result);
echo $num_row;
if( $num_row ==1 )
     {
//Sessie zetten
$_SESSION['user'] = $klantnummer;
header('Refresh: 5; url=index.php');
  }
  else
     {
 $fout = $lang_fout;
  }


//Form sluiten
}




// SYSTEEM INLOGGEN EINDE

Meer tonen

Het zal wel aan mij liggen maar hij leest geen rows, ook al kloppen de gegevens.

bramieboy100

Dan moet je kijken in je database, komen de waarden overeen?

T.Nijborg

Nu heb ik het probleem gefixed qua inloggen etc. En een nieuw topic voor dit probleem vind ik onnodig, ik krijg als je je gegevens wilt aanpassen deze error:

PHP

Warning: Cannot modify header information - headers already sent by (output started at /home/tnijborg/domains/developers4you.nl/public_html/customers/index.php:25) in /home/tnijborg/domains/developers4you.nl/public_html/customers/pages/account.php on line 29

En daarbij gaat het op deze 2 stukjes code:

Lijn 25 uit index:

PHP

<?php echo $menulinks; ?>

Lijn 29 uit account.php

PHP

header('Refresh: 0; url=index.php?page=account&goed=1&security=' . $sessieids . '&save=' . $securityid);

Maar in menulinks, althans die echo, staat niks met header etc, dus waar komt deze fout vandaan...

Victor

Staat er in $menulinks ook niet iets als session_start(); ?

bramieboy100

Er is dan meestal een probleem met ob_start. Probeer deze te verwijderen.

T.Nijborg

Ik heb nergens ob_start staan, en session_start heb ik 1 keer, puur in config.php, voor de inlog, etc

bramieboy100

Probeer dan eens om juist wel ob_start() in je config in te voegen.

T.Nijborg

Okee het werkt nu Dankje.

Mag ik vragen wat ob_start() doet?

ICTscripters

Dé plek voor IT

Dé plek voor IT

Port Father: beta van een browsergame in Antwerpen 1931, feedback welkom

Maffia Hotel: een browsergame waarin je van lobbygast naar spil van het hotel groeit

Crime Club volledig herschreven vanaf nul

Malavita (nieuwe maffia game, van 0)

Herkent deze Notion-structuurscan echte problemen?

🚀 WE ZOEKEN TESTERS! 🚀

Luchtvaartsoftware: Hoe technologie de moderne luchtvaart verandert

Hoe gebruiken jullie WhatsApp voor marketing en klantopvolging?

[TE KOOP] Malavita.be [Maffiaspel, gebaseerd op omicidio.nl, unieke code]

TK: Kraai.eu

[TE KOOP] Likeboost24.nl: complete SMM-webshop met eigen merk, volautomatische levering en betalen op de eigen pagina

Lijst ene klantnummer wel, andere niet.

Registratie

PHP waarschuwt voor twee ernstige bugs die makkelijk uit te buiten lijken

Functioneel ontwerp

Access Control List implementatie in PHP/MySQL - deel 1/2

Access Control List implementatie in PHP/MySQL - deel 2/2

Gebruikers die dit topic bekijken