Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: Uprawnienia po zalogowaniu
Forum PHP.pl > Forum > PHP
bar_kow
27.11.2011, 18:16:02
Witam, męczę się jakiś czas. Może Wy mi pomożecie.
Mam prosty system logowania. Chciałbym do niego dodać odpowiednie uprawnienia. W MySQL dodałem kolumnę uprawnienia, gdzie domyślna wartość to "zwykle".
Jak zrobić aby po zalogowaniu, oprócz nazwy użytkownika pobierał też uprawnienia?

Plik logowania:
[PHP] pobierz, plaintext 
  1. session_start(); 
  2.  
  3. session_destroy();
  4.     $message=""; 
  5.  
  6. // Sekcja logowania
  7. $login=$_POST['login'];
  8. if($login) {
  9.  
  10.   $uzytkownik=$_POST['uzytkownik'];
  11.   $md5_haslo=md5($_POST['haslo']);
  12.  
  13.  
  14.  
  15.   $host="localhost";
  16.   $db_user="UZYTKOWNIK";
  17.   $db_password="HASLO"; 
  18.   $database="NAZWA"; 
  19.     mysql_connect($host,$db_user,$db_password);
  20.     mysql_select_db($database);
  21.  
  22.   $rezultat=mysql_query("select * FROM logowanie WHERE uzytkownik='$uzytkownik' AND haslo='$md5_haslo'");
  23.     if(@mysql_num_rows($rezultat)){
  24.    session_register("uzytkownik");
  25.  
  26.     header("location:main.php"); 
  27.   exit;
  28.   }else {
  29.   $message="<center><p algin='center'><div class='warning_box'>
  30.         Nieprawid?owa nazwa u?ytkownika lub has?o!.
  31.      </div></p></center>";
  32.   }
  33. } 
  34. ?>
  35.  
  36. STRONA
[PHP] pobierz, plaintext


Plik strony (sesja):
[PHP] pobierz, plaintext 
  1. <?
  2.  
  3. session_start(); 
  4.   if(!session_is_registered("uzytkownik")){ 
  5.   header("location:index.php"); 
  6. }
  7. ?>
  8.  
  9. STRONA
[PHP] pobierz, plaintext


Proszę o pomoc...

Edit:

Pobieranie nazwy użytkownika np.:
[PHP] pobierz, plaintext 
  1. Witaj,  <? echo $_SESSION['uzytkownik']; ?>.
[PHP] pobierz, plaintext


Myślałem o tym $_SESSION['uprawnienia'] ale nie działa...;
szmerak
27.11.2011, 18:44:59
Po pierwsze to zabezpiecz te $_POSTy... Twój skrypt jest podatny na SQL INJECTION
a uprawnienia... w SELECie masz gwiazdke, więc pobiera wszystkie kolumny... tak więc co to za problem w indexie tablicy odwolac sie do tej kolumny?
i przypisac ja do zmiennej session

$result[nazwa_kolumny]
modern-web
27.11.2011, 19:20:47
Dodatkowo filtruj dane; ogranicz długość ciągu wprowadzanego do programu oraz zdefiniuj odpowiedni typ zmiennej (np. jeżeli ma być to liczbą to "(int) $zmienna = 1"). Odnośnie SQL Injection - skorzystaj z mysql_real_escape_string. I tak na przyszłość, odradzam korzystania z MySQL. Poczytaj trochę o MySQLi oraz PDO. Nie dość, że szybsze to jeszcze oferują lepszą obsługę błędów smile.gif
bar_kow
27.11.2011, 19:33:50
Coś takiego w pliku strony dać?
$_SESSION['uprawnienia'] = $result[uprawnienia]
?
modern-web
27.11.2011, 19:37:53
W pliku logowania musisz przekazać wartości do sesji - tutaj robisz $_SESSION['uprawnienia'] = $result['uprawnienia'].
Następnie w pliku strony musisz sprawdzić, czy sesja o nazwie 'uprawnienia' istnieje, a potem możesz sobie to przypisać do jakiejś zmiennej ale możesz też operować na zmiennej sesji smile.gif
bar_kow
27.11.2011, 19:46:01
Coś mi dziś nie wychodzi... jakiegoś doła mam. Nawet nie wiem gdzie to wstawić aby działało sad.gif
szmerak
27.11.2011, 19:55:09
Masz pliki... logowanie.php <- odpowiedzialny za sprawdzenie czy uzytkownik podał prawidłowe dane...
i index.php <- gdy użytkownik jest zalogowany ma dostęp do np. ukrytego menu...

W każdym pliku zaczynasz sesje
session_start();

Następnie w pliku logowanie po zwalidowaniu wszystkiego ustawiasz zmienne sesyjne.

$_SESSION['is_user_logged'] = TRUE;
+
$_SESSION['user_name'] = $result[username];
+
$_SESSION['rights'] = $result[rights];

i przekierowywujesz użytkownika na stronę index.php funkcją header();

W pliku index.php musisz sprawdzić czy użytkownik jest zalogowany..
np.
session_start();
<html>
<body>
//treść dostępna dla wszystkich

i tutaj sprawdzasz czy uzytkownik jest zalogowany
if($_SESSION['is_user_logged'] == TRUE) {
//treść tylko dla zalogowanych
Twoja nazwa użytkownika to: $_SESSION['user_name'];
Twoje uprawnienia to: $_SESSION['rights'];
}

//treść dostępna dla wszystkich
</html>

i w taki sposób możesz wszędzie sprawdzać czy użytkownik jest zalogowany...
if($_SESSION['is_user_logged'] == true)
{ tutaj dla zalogowanych }

Równierz wszystkie akcje dodawania/usuwania/edytowania możesz kontrować właśnie w taki sposób dając tylko dostęp zalogowanym użytkownikom...

Aby się wylogować używasz później funkcji session_destroy();
modern-web
27.11.2011, 19:57:30
LOGOWANIE:
[PHP] pobierz, plaintext 
  1. <?php
  2. session_start(); 
  3. $login = mysql_real_escape_string(trim($_POST['login']));
  4. if($login) {
  5.   $uzytkownik = $_POST['uzytkownik'];
  6.   $md5_haslo = md5(mysql_real_escape_string(trim($_POST['haslo'])));
  7.  
  8.   $host="localhost";
  9.   $db_user="UZYTKOWNIK";
  10.   $db_password="HASLO"; 
  11.   $database="NAZWA";
  12.  
  13.     mysql_connect($host, $db_user, $db_password);
  14.     mysql_select_db($database);
  15.  
  16.   $rezultat = mysql_query("SELECT * FROM `logowanie` WHERE `uzytkownik` = '$uzytkownik' AND `haslo` = '$md5_haslo'");
  17.     if (mysql_num_rows($rezultat)) {
  18. 		$_SESSION['uzytkownik'] = $rezultat['uzytkownik'];
  19. 		$_SESSION['uprawnienia'] = $rezultat['uprawnienia'];
  20. 		header("location:main.php"); 
  21.   		exit;
  22.   	} else {
  23. 		$message="<center><p algin='center'><div class='warning_box'>Nieprawid?owa nazwa u?ytkownika lub has?o!.</div></p></center>";
  24.   }
  25. } 
  26. ?>
[PHP] pobierz, plaintext


STRONA:
[PHP] pobierz, plaintext 
  1. <?php
  2. session_start(); 
  3.   	if ( isset($_SESSION['uzytkownik']) && isset($_SESSION['uprawnienia']) ) { 
  4.   		echo 'ok'; 
  5. 	} else {
  6. 		header("location:index.php");
  7. 	}
  8. ?>
[PHP] pobierz, plaintext


Spróbuj teraz smile.gif

Edit - @szmerak
Ja radziłbym wcześniej wyczyścić zawartość tablicy sesyjnej smile.gif $_SESSION = array();
szmerak
27.11.2011, 20:04:07 
[PHP] pobierz, plaintext 
  1. $start_user_sess = $_SESSION['uzytkownik'] = $rezultat('uzytkownik');
  2. $start_user_priv = $_SESSION['uprawnienia'] = $rezultat('uprawnienia');
[PHP] pobierz, plaintext

sorki ale nie wiem po co to skoro przekierowywujesz użytkownika na main.php
a po drugie dałeś nawiasy zwykłe zamiast kwadratowych

@EDIT
@up jak najbardziej zgadzam się z tym smile.gif. Ja mam do tego specjalne funkcje które są dołączane na początku każdego pliku smile.gif
bar_kow
27.11.2011, 20:04:37
W pliku logowania wywala błąd w linii 3
modern-web
27.11.2011, 20:07:45
Faktycznie szmerak! Dzięki za czujność smile.gif Z pośpiechu popełniłem tak radykalny błąd.
Przypisałem te operacje do zmiennych z jednego powodu - żeby wiedział co się w tym momencie odbywa bo widzę, że facet jest ciemny w temacie sesje wink.gif

Treść błędu poproszę.
szmerak
27.11.2011, 20:09:06
Mysql_real_escape_string możesz użyć dopiero po połączeniu się z bazą!

Przenieś to 
[PHP] pobierz, plaintext 
  1.  $host="localhost";
  2.   $db_user="UZYTKOWNIK";
  3.   $db_password="HASLO"; 
  4.   $database="NAZWA";
  5.  
  6.     mysql_connect($host, $db_user, $db_password);
  7.     mysql_select_db($database);
[PHP] pobierz, plaintext


Na samą górę pliku
bar_kow
27.11.2011, 20:10:16
Warning: mysql_real_escape_string() [function.mysql-real-escape-string]: Access denied for user 'coś'@'localhost' (using password: NO) in /home/xxxx/domains/xxxxx.eu/public_html/xxxxx/index.php on line 3

Warning: mysql_real_escape_string() [function.mysql-real-escape-string]: A link to the server could not be established in /home/xxx/domains/xxxx.eu/public_html/xxxxx/index.php on line 3
modern-web
27.11.2011, 20:10:18
Najlepiej wywal poza katalog publiczny strony gdzieś katalog wyżej i użyj require do zaimportowania.
Mayka
27.11.2011, 20:11:44
No bo to że musisz wpisać swoje dane do logowania do sql'a to oczywiście wiesz wink.gif
szmerak
27.11.2011, 20:13:27
Zrób tak weź utwórz osobny plik connect.php daj tam funkcję łączące się z bazą
a tam gdzie potrzebujesz łączyć się z bazą daj require './connect.php'; na samej górze

gotowca chyba nie musze ci dawać?

tak btw... taką masz nazwę użytkownika? Access denied for user 'coś'@'localhost'

"coś"?
modern-web
27.11.2011, 20:15:44
A skąd pewność, że nie jest to subdomena ;>? W tym wypadku pozostaje luka bezpieczeństwa przez domenę. Lepiej żeby wywalił ten katalog/plik ZUPEŁNIE poza public_html (w przypadku XAMPPA - poza htdocs)

@ nie używa się polskich znaków w programowaniu - wbij to sobie do głowy smile.gif
bar_kow
27.11.2011, 20:16:43
Mayka, wiem wiem
Wywalił błąd w linii 23, poprawiłem nawiasy z ( ) na [ ] błąd zniknął. Teraz mam błąd w pliku strony
Warning: session_start() [function.session-start]: Cannot send session cache limiter - headers already sent (output started at

Linia 4:
session_start();

Warning: Cannot modify header information - headers already sent by (output started at /ho

Linia 8:
header("location:index.php");
modern-web
27.11.2011, 20:17:48
dodaj

ob_start();

w 2 linijce kodu (pod session_start()wink.gif i

ob_end_flush();

na końcu pliku
szmerak
27.11.2011, 20:19:02
To znaczy że nagłówki zostały już wysłane...
Musisz wykonać te funkcje przed wyświetleniem czegokolwiek!
bar_kow
27.11.2011, 20:21:54
Jak zrobiłem tak:
[PHP] pobierz, plaintext 
  1.     <?php
  2.     session_start();
  3.  
  4. ob_start();
  5.  
  6.     if ( isset($_SESSION['uzytkownik']) && isset($_SESSION['uprawnienia']) ) {
  7.     echo 'ok';
  8.     } else {
  9.     header("location:index.php");
  10.     }
  11.     ?>
[PHP] pobierz, plaintext

STRONA
[PHP] pobierz, plaintext 
  1. </div>
  2. <?PHP
  3. require('stopka.php');
  4. ob_end_flush();
  5. ?>
[PHP] pobierz, plaintext


To wywala błędy w liniach:
4:
Warning: session_start() [function.session-start]: Cannot send session cache limiter - headers already sent

oraz 11
Warning: Cannot modify header information - headers already sent by (output started at
czyli w lini 11 jest header("location:index.php");
modern-web
27.11.2011, 20:24:40
Odpowiem tak; nie chce mi się tego wszystkiego tłumaczyć jak wątek ten był poruszany 1000 razy.
Rzuć okiem tutaj -> http://forum.ovh.pl/showthread.php?t=7960
i tutaj -> http://pl.wikibooks.org/wiki/PHP/Sesje
oraz tutaj -> header
szmerak
27.11.2011, 20:27:06
Przed znakiem <? masz spacje... A tam jej nie może być ponieważ ta spacja jest wyświetlana przed startem sesji.
Pamiętaj że te funkcje muszą być wywołane przed wysłaniem nagłówków...

Usuń spacje i powinno działać
bar_kow
27.11.2011, 20:30:03
Teraz nawet nie chce zalogować...
Ok wrócę do tego za tydzień, bo jutro wyjeżdżam na tygodniową konferencję po za Polskę...
modern-web
27.11.2011, 20:30:56
Język PHP pomija przecież spacje ;|, a na dodatek mówisz tu o spacjach, które wykraczają poza obszar skryptu...
szmerak
27.11.2011, 20:34:21
Ale przecież to co nie jest w znacznikach <?php?> jest wyświetlane jako HTML? right?

Tak więc to był problem ponieważ HTML(tutaj spacje) zostały wyświetlone przed wykonaniem kodu php

Nie potrzebnie to ciągniemy bar_kow daj kod źródłowy... Zobacze czy jestem wstanie ci pomóc
bar_kow
27.11.2011, 20:42:04
Plik index.php
[PHP] pobierz, plaintext 
  1. <?php 
  2.  
  3. session_start();
  4.  
  5. $host="localhost"; // Nazwa hosta.
  6.   $db_user="nazwa_1"; // Nazwa użytkownika - MySQL.
  7.   $db_password="haslo"; // Hasło do bazy.
  8.   $database="tabela"; // Nazwa bazy.
  9.     mysql_connect($host,$db_user,$db_password);
  10.     mysql_select_db($database);
  11. $login = mysql_real_escape_string(trim($_POST['login']));
  12. if($login) {
  13. $uzytkownik = $_POST['uzytkownik'];
  14. $md5_haslo = md5(mysql_real_escape_string(trim($_POST['haslo'])));
  15.  
  16.  
  17. // Połączenie z bazą.
  18.  
  19.  
  20.  
  21.  
  22.  
  23.     $rezultat = mysql_query("SELECT * FROM logowanie WHERE uzytkownik = '$uzytkownik' AND haslo = '$md5_haslo'");
  24. if (mysql_num_rows($rezultat)) {
  25. $start_user_sess = $_SESSION['uzytkownik'] = $rezultat['uzytkownik'];
  26. $start_user_priv = $_SESSION['uprawnienia'] = $rezultat['uprawnienia'];
  27. header("location:main.php");
  28. exit;
  29. } else {
  30. $message="<center><p algin='center'><div class='warning_box'>Nieprawid?owa nazwa u?ytkownika lub has?o!.</div></p></center>";
  31. }
  32. }
  33. ?>
  34.  
  35.  
  36.  
  37.  
  38.  
  39. <!-- PAGE --!>
  40.  
  41.  
  42. <?PHP
  43. require('gora.php');
  44. ?>
  45.  
  46.  
  47.     <div id="templatemo_main_top"></div>
  48.     <div id="templatemo_main"><span id="main_top"></span><span id="main_bottom"></span>
  49.  
  50. <?PHP
  51. require('bok.php');
  52. ?>
  53. <?PHP
  54. require('../info.php');
  55. ?>
  56.  
  57.  
  58.  
  59.             <div class="cleaner"></div>
  60.         </div> <!-- end of sidebar -->
  61.  
  62.         <div id="templatemo_content">
  63.  
  64.             <div class="content_box">
  65.             	<h2>Logowanie</h2>
  66.  
  67.           </div>
  68. <div class="content_box">
  69.  
  70.  
  71.  
  72.                  <form id="form1" name="form1" method="post" action="<? echo $PHP_SELF; ?>">
  73.  
  74.  
  75.                 Username:<br>
  76.                         <input type="text" name="uzytkownik"  id="uzytkownik" size="35" /><br>
  77.                         Password:<br>
  78.                         <input type="password" name="haslo" id="haslo" size="35" /><br>
  79.  
  80.                     <input type="submit" name="login" id="login" value="Enter" />
  81.  
  82.  
  83.          </form>
  84.  
  85.                 <div class="cleaner"></div>
  86.  
  87.             </div>
  88.  
  89.             </div>
  90.  
  91.         <div class="cleaner"></div>    
  92.     </div>
  93.  
  94.     <div id="templatemo_main_bottom">
  95.     </div>
  96.  
  97. </div> <!-- end of wrapper -->
  98. </div>
  99. <?PHP
  100. require('stopka.php');
  101.  
  102. ?>
[PHP] pobierz, plaintext


Plik main.php

[PHP] pobierz, plaintext 
  1. <?php
  2. session_start();    
  3.  
  4.  
  5.     if ( isset($_SESSION['uzytkownik']) && isset($_SESSION['uprawnienia']) ) {
  6.     echo 'ok';
  7.     } else {
  8.     header("location:index.php");
  9.     }
  10.     ?>
  11.  
  12.  
  13. <!-- PAGE --!>
  14.  
  15.  
  16. <?PHP
  17. require('goraz.php');
  18. ?>
  19.  
  20.  
  21.     <div id="templatemo_main_top"></div>
  22.     <div id="templatemo_main"><span id="main_top"></span><span id="main_bottom"></span>
  23.  
  24. <?PHP
  25. require('bokz.php');
  26. ?>
  27. <?PHP
  28. require('../info.php');
  29. ?>
  30.  
  31.  
  32.  
  33.             <div class="cleaner"></div>
  34.         </div> <!-- end of sidebar -->
  35.  
  36.         <div id="templatemo_content">
  37.  
  38.             <div class="content_box">
  39.             	<h2>Witaj <? echo $_SESSION['uzytkownik']; ?> | Uprawnienia: <? echo $_SESSION['uprawnienia']; ?>  </h2>
  40.  
  41.           </div>
  42. <div class="content_box">
  43.             	Aby kontynuować, wybierz odpowiednią pozycję w menu.
  44.                 <div class="cleaner"></div>
  45.  
  46.             </div>
  47.  
  48.             </div>
  49.  
  50.         <div class="cleaner"></div>    
  51.     </div>
  52.  
  53.     <div id="templatemo_main_bottom">
  54.     </div>
  55.  
  56. </div> <!-- end of wrapper -->
  57. </div>
  58. <?PHP
  59. require('stopka.php');
  60.  
  61. ?>
  62.  
  63.  
  64.  
  65.  
  66.  
  67.  
[PHP] pobierz, plaintext


Juz tak namieszałem że sam już nic nie rozumiem, a głowa pi pęka przez stres... makabra
szmerak
27.11.2011, 21:16:01
Masz tutaj pliki.
index.php http://wklej.to/KyhwW
main.php http://wklej.to/TCbYN
connect.php http://wklej.to/ay9l5
zrobiłem na świeżo... niesprawdzałem

Używasz jakiegoś notepada? do pisania kodu?
bo masz straszny bałagan tam!

Dam ci jedną rade gdy robisz instrukcję if, funkcje, pętle ogólnie wszystko co otwiera i zamyka blok { }
Staraj się robić wcięcia tabulatur w środku... bardzo poprawia to czytelność kodu

i ogólnie zablokuj możliwość wejścia na formularz logowania jeśli ktoś jest zalogowany (zapomniało mi się)

daj na górze if($_SESSION['auth'] == TRUE) die("Wyloguj sie pierwsze");

Jeszcze jedno jak pewnie zauważyłeś użyłem ukrytego pola <input type="hidden" name="sent">
Jest to dobry sposób na sprawdzanie czy formularz został wysłany
bar_kow
27.11.2011, 21:24:14
Wielkie dzięki!
Tak, używam PSPad'a
Jednak nie pokazuje uprawnień sad.gif tylko nazwę użytkownika sad.gif
szmerak
27.11.2011, 21:27:04
EDIT!
Omg jak ja mogłem tego nie zauważyć... Tam jest tylko zapytanie nie ma użytej funkcji mysql_fetch_array();
bar_kow
27.11.2011, 21:29:51
$rezultat = mysql_query("SELECT uzytkownik, uprawnienia FROM logowanie WHERE uzytkownik = '$uzytkownik' AND haslo = '$md5_haslo'");

nadal nie pokazuje uprawnień...
<h2>Witaj <? echo $_SESSION['uzytkownik']; ?> | Uprawnienia: <? echo $_SESSION['uprawnienia']; ?> </h2>

Pokazuje TYLKO nazwę użytkownika
szmerak
27.11.2011, 21:35:30
To:
[PHP] pobierz, plaintext 
  1. $_SESSION['uzytkownik'] = $rezultat['uzytkownik'];
  2. 		$_SESSION['uprawnienia'] = $rezultat['uprawnienia'];
  3. 		$_SESSION['auth'] = TRUE;
[PHP] pobierz, plaintext


zamień na:
[PHP] pobierz, plaintext 
  1. 	$dane = mysql_fetch_array($rezultat);
  2. 		$_SESSION['uzytkownik'] = $dane['uzytkownik'];
  3. 		$_SESSION['uprawnienia'] = $dane['uprawnienia'];
  4. 		$_SESSION['auth'] = TRUE;
[PHP] pobierz, plaintext


Sorki mój błąd nie zauważyłem że tam jest tylko zapytanie mysql_query();
bar_kow
27.11.2011, 21:37:36
Działa! Dziękuję bardzo!!!
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.