Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [PHP]Przerobienie logowania
Forum PHP.pl > Forum > Przedszkole
buliyo
9.05.2009, 21:17:36
Witam znalazłem kiedyś na tym forum taki zgrabny skrypcik do logowania sie na sesjach z userem w mysql..
[PHP] pobierz, plaintext 
  1. <?
  2. ob_start();
  3. ob_start();
  4. require_once('./config.php');
  5. session_start();
  6. if($_SESSION['access'] != 'root'){
  7. @header ("Location:  index.php");
  8. }
  9. ob_end_flush();
  10. function usun($data)
  11. {
  12. GLOBAL $dbc;
  13.   if (ini_get('magic_quotes_gpc'))
  14.   {
  15.       $data = stripslashes($data);
  16.   }
  17. return mysql_real_escape_string($data, $dbc);
  18. }
  19.   if (isset($_POST['submit'])){
  20.   if (empty($_POST['login'])){
  21.       $message .= '<font class="blad">Nie podałeś loginu</font><br />';
  22.       $l = FALSE;}
  23.   else
  24.   {
  25.   $l = usun($_POST['login']);
  26.   }
  27.   if (empty($_POST['haslo']))
  28.   {
  29.   $message .= '<font class="blad">Nie podałeś hasła</font><br />';
  30.   $h = FALSE;
  31.   }
  32.   else
  33.   {
  34.   $h = usun($_POST['haslo']);
  35.   }
  36.   if ( $l && $h )
  37.   {
  38.   $query = "SELECT username, access FROM user WHERE username='$l' AND haslo=password('$h')";
  39.   $result = mysql_query($query);
  40.   $row = mysql_fetch_array ($result, MYSQL_ASSOC);
  41.   if($row)
  42.   {
  43.   session_start();
  44.   $_SESSION['username'] = $row['username'];
  45.   $_SESSION['access'] = $row['access'];
  46.   $_GET['cmd'] = 'ok';
  47.   header ("Location: list.php");  // przekierowuje nas na jaka strone chcemy
  48.   }
  49.   else
  50.   {
  51.   $message .= '<font class="blad">Błędne hasło lub login</font><br />';
  52.   }
  53.   mysql_close();
  54.   }
  55.   else
  56.   {
  57.   $message .= '<font class="blad">Spróbuj jeszcze raz</font>';
  58.   }
  59. }
  60. ob_end_flush();
  61. ?>
  62. <div align="center">
  63. <? if(isset($message)){
  64. echo $message;
  65. }
  66. ?>
  67. </div>
[PHP] pobierz, plaintext


chciałem go przerobić aby user i haslo bylo na sztywno w kodzie, zrobilem cos takiego:
[PHP] pobierz, plaintext 
  1. <?
  2. ob_start();
  3. ob_start();
  4. session_start();
  5. if($_SESSION['access'] != 'root'){
  6. @header ("Location:  index.php");
  7. }
  8. ob_end_flush();
  9. function usun($data)
  10. {
  11. GLOBAL $dbc;
  12.   if (ini_get('magic_quotes_gpc'))
  13.   {
  14.       $data = stripslashes($data);
  15.   }
  16. return mysql_real_escape_string($data, $dbc);
  17. }
  18.   if (isset($_POST['submit'])){
  19.   if (empty($_POST['login'])){
  20.       $message .= '<font class="blad">Nie podałeś loginu</font><br />';
  21.       $l = FALSE;}
  22.   else
  23.   {
  24.   $l = usun($_POST['login']);
  25.   }
  26.   if (empty($_POST['haslo']))
  27.   {
  28.   $message .= '<font class="blad">Nie podałeś hasła</font><br />';
  29.   $h = FALSE;
  30.   }
  31.   else
  32.   {
  33.   $h = usun($_POST['haslo']);
  34.   }
  35.   {
  36.   session_start();
  37.   $_SESSION['username'] = 'admin';
  38.   $_SESSION['access'] = 'admin';
  39.   $_GET['cmd'] = 'ok';
  40.   header ("Location: tabelki.php");  // przekierowuje nas na jaka strone chcemy
  41.   }
  42.   else
  43.   {
  44.   $message .= '<font class="blad">Błędne hasło lub login</font><br />';
  45.   }
  46.   }
  47.   else
  48.   {
  49.   $message .= '<font class="blad">Spróbuj jeszcze raz</font>';
  50.   }
  51. }
  52. ob_end_flush();
  53. ?>
  54. <div align="center">
  55. <? if(isset($message)){
  56. echo $message;
  57. }
  58. ?>
  59. </div>
[PHP] pobierz, plaintext

ale niestety wywala

Parse error: syntax error, unexpected T_ELSE in /login.php on line 42
czy ogolnie moj zamysl jest dobry tylko mam gdzies blad ze skladnia?
wizard144
9.05.2009, 21:48:13
Cytat(buliyo @ 9.05.2009, 22:17:36 ) *
Witam znalazłem kiedyś na tym forum taki zgrabny skrypcik do logowania sie na sesjach z userem w mysql..
[php]

else
{
$h = usun($_POST['haslo']);
}
{ /// <<<<<<<<<<<<<<<<<<<<<<<<<<<<< Tu błąd
session_start();
$_SESSION['username'] = 'admin';
$_SESSION['access'] = 'admin';
$_GET['cmd'] = 'ok';
header (\"Location: tabelki.php\"); // przekierowuje nas na jaka strone chcemy
}


po 1. brak warunku if
po 2. ob_start masz 2 razy... po co ?

a co do Twojej ideologii hmmm... przydało by się sprawdzić czy zmienna $_POST['login'], $_POST['haslo'] jest równa czemuś nie uważasz ? =] i wtedy ustawiać zmienne sesyjne dopiero smile.gif

Edit :

To właśnie ten warunek if smile.gif if($_POST['login'] == 'admin' && $_POST['haslo'] == 'admin') i tu masz ten błąd 42 linii smile.gif
buliyo
10.05.2009, 03:38:39 
[PHP] pobierz, plaintext 
  1. <?
  2. ob_start();
  3. ob_start();
  4. session_start();
  5. if($_SESSION['access'] != 'root'){
  6. @header ("Location:  index.php");
  7. }
  8. ob_end_flush();
  9. function usun($data)
  10. {
  11. GLOBAL $dbc;
  12.   if (ini_get('magic_quotes_gpc'))
  13.   {
  14.       $data = stripslashes($data);
  15.   }
  16. return mysql_real_escape_string($data, $dbc);
  17. }
  18.   if (isset($_POST['submit'])){
  19.   if (empty($_POST['login'])){
  20.       $message .= '<font class="blad">Nie podałeś loginu</font><br />';
  21.       $l = FALSE;}
  22.   else
  23.   {
  24.   $l = usun($_POST['login']);
  25.   }
  26.   if (empty($_POST['haslo']))
  27.   {
  28.   $message .= '<font class="blad">Nie podałeś hasła</font><br />';
  29.   $h = FALSE;
  30.   }
  31.   else
  32.   {
  33.   $h = usun($_POST['haslo']);
  34.   }
  35.   if ( $l && $h )
  36.   {
  37.   if($_POST['login'] == 'admin' && $_POST['haslo'] == 'admin')
  38.   {
  39.   session_start();
  40.   $_SESSION['username'] = 'admin';
  41.   $_SESSION['access'] = 'admin';
  42.   $_GET['cmd'] = 'ok';
  43.   header ("Location: tabelki.php");  // przekierowuje nas na jaka strone chcemy
  44.   }
  45.   else
  46.   {
  47.   $message .= '<font class="blad">Błędne hasło lub login</font><br />';
  48.   }
  49.   mysql_close();
  50.   }
  51.   else
  52.   {
  53.   $message .= '<font class="blad">Spróbuj jeszcze raz</font>';
  54.   }
  55. }
  56. ob_end_flush();
  57. ?>
  58. <div align="center">
  59. <? if(isset($message)){
  60. echo $message;
  61. }
  62. ?>
  63. </div>
[PHP] pobierz, plaintext


Po takiej modyfikacji skryptu, błąd nie wyświetla się ale niestety po wpisaniu hasła i loginu jesteśmy jeszcze raz kierowani na stronę index.php, sesja nie jest otwierana ponieważ ręczne wejście na tabelki.php też zwraca nas na index.php
kwiateusz
10.05.2009, 10:29:37
rozumiem że w skrypcie ktory kieruje do tego podanego wyżej ustawiasz $_SESSION['access'] na root?
buliyo
10.05.2009, 10:40:17 
[PHP] pobierz, plaintext 
  1. <?
  2. ob_start();
  3.  
  4. session_start();
  5. if($_SESSION['access'] != 'root'){
  6. @header ("Location:  index.php");
  7. }
  8. ob_end_flush();
  9. ?>
[PHP] pobierz, plaintext


no tak mam zabezpieczona strone tabelki.php
o to chodz?

na stronie kierujacej czy lindex.php mam taki formularz:
[HTML] pobierz, plaintext 
  1. <FORM ACTION="login.php" METHOD="POST">
  2.   <CENTER><div class="moje">login:<BR>
  3.   <INPUT TYPE="TEXT" NAME="login" size=10><BR>
  4.   hasło:<BR>
  5.   <INPUT TYPE="PASSWORD" NAME="haslo" size=10><BR>
  6.   <INPUT TYPE="SUBMIT" name="submit" VALUE="Akceptuj"></div></CENTER>
  7.   </FORM>
[HTML] pobierz, plaintext
kwiateusz
10.05.2009, 10:53:45
na stronie kierujacej tez musisz mieć session_start(), a potem $_SESSION['access'] = 'root' bo niby skąd taka zmienna mas się objawić? nie ma to skrypt z ifa zwraca true i jestes przekierowywane na index.php
buliyo
10.05.2009, 11:01:12
Po wrzuceniu na index.php
[PHP] pobierz, plaintext 
  1. <?
  2. ob_start();
  3.  
  4. session_start();
  5. if($_SESSION['access'] != 'root'){
  6. @header ("Location:  index.php");
  7. }
  8. //phpinfo();
  9. ob_end_flush();
  10. ?>
[PHP] pobierz, plaintext

Pętla przekierowań

Firefox wykrył, że serwer przekierowuje żądanie tego zasobu w sposób uniemożliwiający jego ukończenie.

Przeglądarka przerwała próby pobrania żądanego elementu. Witryna przekierowuje żądanie w sposób, który uniemożliwia jego dokończenie.

* Czy ciasteczka zostały wyłączone lub zablokowane dla tej witryny?
* UWAGA: Jeśli włączenie obsługi ciasteczek dla tej witryny nie rozwiązuje problemu, najprawdopodobniej jest to problem w konfiguracji serwera, a nie oprogramowania na Twoim komputerze.
kwiateusz
10.05.2009, 11:22:21
Ty zacznij czytac ze zrozumieniem, czy ja gdzieś napisałem że masz całego ifa przekleić? nie... więc czytaj jeszcze raz
buliyo
10.05.2009, 11:47:42
Cytat(kwiateusz @ 10.05.2009, 09:29:37 ) *
rozumiem że w skrypcie ktory kieruje do tego podanego wyżej ustawiasz $_SESSION['access'] na root?


jeszcze raz...
mam obecnie cos takiego:
index.php
[PHP] pobierz, plaintext 
  1. <?
  2. ob_start();
  3. session_start();
  4. $_SESSION['access'] = 'root';
  5. ob_end_flush();
  6. ?>
  7. <!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
  8. <html><head>
  9.  
  10. <TITLE>logowanie</TITLE>
  11. <META HTTP-EQUIV="Content-type" CONTENT="text/html; charset=iso-8859-2">
  12. <link rel="stylesheet" type="text/css" href="styl.css">
  13. </HEAD>
  14. <BODY BGCOLOR="#434242">
  15. <FORM ACTION="login.php" METHOD="POST">
  16.   <CENTER><div class="moje">login:<BR>
  17.   <INPUT TYPE="TEXT" NAME="login" size=10><BR>
  18.   hasło:<BR>
  19.   <INPUT TYPE="PASSWORD" NAME="haslo" size=10><BR>
  20.   <INPUT TYPE="SUBMIT" name="submit" VALUE="Akceptuj"></div></CENTER>
  21.   </FORM>
  22.  
  23. </BODY>
  24. </HTML>
[PHP] pobierz, plaintext

login.php
[PHP] pobierz, plaintext 
  1. <?
  2. ob_start();
  3. session_start();
  4. if($_SESSION['access'] != 'root'){
  5. @header ("Location:  index.php");
  6. }
  7. ob_end_flush();
  8. function usun($data)
  9. {
  10. GLOBAL $dbc;
  11.   if (ini_get('magic_quotes_gpc'))
  12.   {
  13.       $data = stripslashes($data);
  14.   }
  15. }
  16.   if (isset($_POST['submit'])){
  17.   if (empty($_POST['login'])){
  18.       $message .= '<font class="blad">Nie podałeś loginu</font><br />';
  19.       $l = FALSE;}
  20.   else
  21.   {
  22.   $l = usun($_POST['login']);
  23.   }
  24.   if (empty($_POST['haslo']))
  25.   {
  26.   $message .= '<font class="blad">Nie podałeś hasła</font><br />';
  27.   $h = FALSE;
  28.   }
  29.   else
  30.   {
  31.   $h = usun($_POST['haslo']);
  32.   }
  33.   if ( $l && $h )
  34.   {
  35.   if($_POST['login'] == 'admin' && $_POST['haslo'] == 'admin')
  36.   {
  37.   session_start();
  38.   $_SESSION['username'] = 'admin';
  39.   $_SESSION['access'] = 'admin';
  40.   $_GET['cmd'] = 'ok';
  41.   header ("Location: tabelki.php");  // przekierowuje nas na jaka strone chcemy
  42.   }
  43.   else
  44.   {
  45.   $message .= '<font class="blad">Błędne hasło lub login</font><br />';
  46.   }
  47.   mysql_close();
  48.   }
  49.   else
  50.   {
  51.   $message .= '<font class="blad">Spróbuj jeszcze raz</font>';
  52.   }
  53. }
  54. ob_end_flush();
  55. ?>
  56. <div align="center">
  57. <? if(isset($message)){
  58. echo $message;
  59. }
  60. ?>
  61. </div>
[PHP] pobierz, plaintext

tabelki.php
[PHP] pobierz, plaintext 
  1. <?
  2. ob_start();
  3.  
  4. session_start();
  5. if($_SESSION['access'] != 'root'){
  6. @header ("Location:  index.php");
  7. }
  8. ob_end_flush();
  9. ?>
  10. <!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
  11. <html><head>
  12.  
  13. <TITLE>tabelki</TITLE>
  14. <META HTTP-EQUIV="Content-type" CONTENT="text/html; charset=iso-8859-2">
  15. <link rel="stylesheet" type="text/css" href="styl.css">
  16. </HEAD>
  17. <BODY BGCOLOR="#434242">
  18. działa!!
  19. </BODY>
  20. </HTML>
[PHP] pobierz, plaintext

po logowaniu przy uzyciu admin/admin otrzymuje
Spróbuj jeszcze raz

sad.gif
wizard144
10.05.2009, 14:17:33 
[PHP] pobierz, plaintext 
  1. <?php
  2. if($_SESSION['access'] != 'root'){
  3. ?>
[PHP] pobierz, plaintext 


[PHP] pobierz, plaintext 
  1. <?php
  2. if($_POST['login'] == 'admin' && $_POST['haslo'] == 'admin')
  3.  {
  4.  session_start();
  5.  $_SESSION['username'] = 'admin';
  6.  $_SESSION['access'] = 'admin';
  7.  $_GET['cmd'] = 'ok';
  8.  header ("Location: tabelki.php");  // przekierowuje nas na jaka strone chcemy
  9.  }
  10. ?>
[PHP] pobierz, plaintext



1. Hasło - > admin Login -> admin | tak ?
2. Skoro definiujesz że przy dobrym podaniu hasła zmienna $_SESSION['acces'] = 'admin'; to przy wejściu do tabelek nie możesz mieć if'a takiego jak masz...

powinno być

[PHP] pobierz, plaintext 
  1. <?php
  2. if($_SESSION['acces'] != 'admin')
  3. {
  4. @header ("Location:  index.php");
  5. }
  6. ?>
[PHP] pobierz, plaintext


Popraw to i powinno działać

EDIT

[PHP] pobierz, plaintext 
  1. <?php
  2. if (empty($_POST['login'])){
  3.      $message .= '<font class="blad">Nie podałeś loginu</font><br />';
  4.      $l = FALSE;}
  5. /* else
  6.  {
  7.  $l = usun($_POST['login']);
  8.  }*/
  9.  if (empty($_POST['haslo']))
  10.  {
  11.  $message .= '<font class="blad">Nie podałeś hasła</font><br />';
  12.  $h = FALSE;
  13.  }
  14. /* else
  15.  {
  16.  $h = usun($_POST['haslo']);
  17.  }*/
  18. ?>
[PHP] pobierz, plaintext


zakomentuj to co ja, przecież po podaniu loginu i hasła jest ono usuwane z tego co widzę
kwiateusz
10.05.2009, 14:34:54
a zobacz do czego służy opcja usun? :] do usuwania s;ashy nie do usuwania hasła
2. a skad wystrzasnąłes $_SESSION['access'] = admin? to nie ma nijakiego związku z $_POST['login'], w pliku index ustala że root i sie zgadza nie wywołuje się pierwszy redirect, wiec o co Ci chodzi wizard?
wizard144
10.05.2009, 19:34:05
Racja mój błąd. Ale w pliku "tabelki.php" jest redirect z warunkiem

[PHP] pobierz, plaintext 
  1. <?php
  2. if($_SESSION['acces'] != 'root')
  3. ?>
[PHP] pobierz, plaintext


A podczas logowania sam ustala że 

[PHP] pobierz, plaintext 
  1. <?php
  2. $_SESSION['acces'] = 'admin'
  3. ?>
[PHP] pobierz, plaintext


Nie ?
kwiateusz
10.05.2009, 20:01:02
a rzeczywiście, my fault kijowy ten skrypt że się tak wyraże smile.gif
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.