login  Naam:   Wachtwoord: 
Registreer je!
 Forum

Inlog Probleem

Offline sicco - 08/08/2005 09:40 (laatste wijziging 08/08/2005 09:41)
Avatar van siccoLid heey mensen, ik heb voor mijn admin een inlog systeem die gebruikt maakt van sessions, alleen als ik inlog en vervolgens door ga naar index.php staat er weer dat ik niet ingelogt ben 

ik heb dit als inloggen.php:

php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?
  4. ?>
  5. <?php
  6. include('config.php');
  7.  
  8. if($submit){
  9.  
  10.  mysql_connect($host,$gebruikersnaam,$wachtwoord);
  12.  
  13.  $sql = "SELECT * FROM admin WHERE ww = '" . $_POST['wachtwoord']. "' AND nickname = '" . $_POST['nickname']. "' LIMIT 1";
  14.  $query = mysql_query($sql) or die(mysql_error());
  15.  
  16.  if (mysql_num_rows($query) == 1){
  17.  $naam = ucfirst($_POST[nickname]);
  18.  
  19.  $_SESSION['admin'] = "$naam";
  20.  echo "Je bent ingelogd "; echo $_SESSION['admin']; echo "!<br>Klik <a href=\"index.php\">hier</a> voor de Admin.";
  21.  }elseif (mysql_num_rows($query) == 0){
  22.  $tekst = "Je bent niet ingelogd!<br>Klik <a href=\"inloggen.php\">hier</a> en probeer het opnieuw!";
  23.  echo "$tekst";
  24. }}else{
  25. ?>
  26. <FORM name="login" METHOD="post" ACTION="<? $PHP_SELF ?>">
  27.   <table width="276" cellspacing="0" cellpadding="0">
  28.     <tr>
  29.       <td width="112">Nickname:</td>
  30.       <td width="162">
  31.         <input type="text" name="nickname" size="25">
  32.       </td>
  33.     </tr>
  34.     <tr>
  35.       <td width="112">Wachtwoord:</td>
  36.       <td width="162">
  37.         <input type="password" name="wachtwoord" size="25">
  38.       </td>
  39.     </tr>
  40.     <tr>
  41.       <td width="112">&nbsp;</td>
  42.       <td width="162">&nbsp;</td>
  43.     </tr>
  44.     <tr>
  45.       <td width="112">&nbsp;</td>
  46.       <td width="162">
  47.         <input type="submit" name="submit" value="Inloggen">
  48.         <input type="reset" name="reset" value="Legen">
  49.       </td>
  50.     </tr>
  51.   </table>
  52.   </FORM>
  53. <?php
  54. }
  55. ?>


en dan vervolgens heb ik boven aan mijn index dit staan

php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?
  2.  
  3. if($_SESSION['admin'] != ""){ 
  4. ?>


en onderaan de pagina

php code - Bekijk de code zonder highlighting - Klap code in 
  1. <? 
  2. }
  3. else{ 
  4. 	echo ("U bent niet ingelogd!<br>");
  5. 	echo ("<a href=\"inloggen.php\">Klik hier</a> om in te loggen."); }
  6. ?>

Het zou toch gewoon moeten merken of zie ik dat nu verkeert

7 antwoorden

Gesponsorde links
Offline Legolas - 08/08/2005 09:50 (laatste wijziging 08/08/2005 09:51)
Avatar van Legolas Onbekend session_start(); helemaal bovenaan bij index.php.

Dat betekent namelijk dat de sessie óf gestart wordt of wordt voortgezet;-)
Offline sicco - 08/08/2005 10:04
Avatar van sicco Lid bedankt, het zijn van die kleine foutjes toch altijd heh 
Offline xSc - 08/08/2005 10:12 (laatste wijziging 10/08/2005 08:17)
Avatar van xSc Onbekend Sicco, ik vind het vervelend om te zeggen, maar dit script is bar slecht.

Om maar eens wat op te noemen:

- Mogelijkheid tot SQL-injection.
- Wachtwoord wordt niet-gecodeerd opgeslagen.
- Niet consequent gebruik van super globals.
- Onjuist quote-gebruik.
- Geen goede foutafhandeling.
- Variabelen tussen quotes.
- ob_start() gebruiken om bijv. onjuist gebruik van een header() te rechtvaardigen, moet je echt afleren. Het is absoluut niet netjes.

Verder is dit geen inlogscript. Een inlogsysteem is meer dan alleen controleren of een waarde overeenkomt en vervolgens een sessie starten.

Tips:
- Sla geen gebruikersnaam / wachtwoord op in een sessie of cookie, maar sla alleen het lid_id op. Aan de hand van het lid_id kun je alles nakijken.
- Gebruik voor het opslaan de functie md5() van PHP. Die gebruik je dan ook voor het vergelijken:

php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php
  2.  
  3. if (!$rLogin = mysql_query("SELECT lid_id FROM leden WHERE wachtwoord = '" . md5($_POST['wachtwoord']) . "'"))
  4. {
  5.  
  6.    echo 'Combinatie gebruikersnaam / wachtwoord onjuist';
  7.  
  8. }
  9. else
  10. {
  11.  
  12.    // juist ingelogd; start sessie
  13.  
  14. }
  15.  
  16. ?>


Natuurlijk bouw je een functie, waarmee je bij elke pagina controleert of de betreffende persoon ingelogd is. Je slaat bijv. het lid_id op, het ip-adres, evt. een controle getal. Mooi is ook om de database erbij te gebruiken. De gegevens in je sessie sla je ook in de database op. Je kunt dan controleren of het lid_id uit de sessie overeenkomt met het lid_id in de database. Ook kun je controleren of het ip-adres in de sessie overeenkomt met het ip-adres in de database het $_SERVER['REMOTE_ADRR']. Hetzelfde geldt natuurlijk voor het eventuele controle-getal. Op deze manier maak je een (behoorlijk) veilig systeem.

Succes!
Offline BigBug - 08/08/2005 10:23
Avatar van BigBug PHP expert @ Henri,

De 1 vindt variabelen buiten quote's beter en de ander vindt variabelen binnen quote's beter.. Wat is nouw het best (reactie van alle leden plz..)
Offline xSc - 08/08/2005 10:31
Avatar van xSc Onbekend @Bigbug, normaal gesproken is het beter om buiten de quotes te houden. Het is trouwens ook sneller.
Offline Gerard - 08/08/2005 11:21
Avatar van Gerard Ouwe rakker Variabelen altijd buiten de quotes houden.
Offline Legolas - 08/08/2005 12:06
Avatar van Legolas Onbekend ik ben het met proximus en henri eens omdat arrays binnen de quotes worden genegeerd, dus das alleen maar lastig. verder is het slordig:s
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.337s