Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
 Reply to this topicStart new topic
> [php]logowanie
-Gość_Leon_*-
post
Post #1





Goście
Mam dziwny problem.
Napisałem skrypt logowania.
Loginy i hasła mam w pliku passwords.txt, w stylu login:hasło każde w osobnej linijce.
Wszystko łądnie ale jak sie loguje to zawsze wyskakuje że złe login lub hasło.
Podaję kod.
Wpisywanie danych:
[HTML] pobierz, plaintext 
  1. <form name = "logowanie"
  2. 	action = "administracja/login.php"
  3. 	methot = "post"
  4. >
  5. Użytkownik:<br><input type="text" name="user"><br>
  6. Hasło:<br><input type="password" name="haslo">
  7. <br><br>
  8. <input type="submit" value="Wejdź">
  9. </form>
[HTML] pobierz, plaintext


Weryfikacja:
[PHP] pobierz, plaintext 
  1. <?php
  2. function checkPass($user, $pass)
  3. {
  4.   if(!$fd = @fopen("passwords.txt", "r")) return 1;
  5.   $result = 2;
  6.   while (!feof ($fd)){
  7. 	$line = trim(fgets($fd));
  8. 	$arr = explode(":", $line);
  9. 	if(count($arr) < 2) continue;
  10.  
  11. 	if($arr[0] != $user) continue;
  12.  
  13. 	if($arr[1] == $pass){
  14. 	  $result = 0;
  15. 	}
  16. 	break;
  17.   }
  18.   fclose($fd);
  19.   return $result;
  20. }
  21. if(!isSet($_POST["haslo"]) || !isSet($_POST["user"])){
  22.   include('bad_login.html');
  23.   exit();
  24. }
  25. $val = checkPass($_POST["user"], $_POST["haslo"]);
  26. switch($val){
  27.   case 0 : include('c5jkvc.php');break;
  28.   case 1 : include('error_server.html');break;
  29.   case 2 : include('bad_login.html');break;
  30.   default : include('error_server.html');
  31. }
  32. ?>
[PHP] pobierz, plaintext


Co robie źle?
Go to the top of the page
+Quote Post
suck
post
Post #2





Grupa: Zarejestrowani
Postów: 43
Pomógł: 0
Dołączył: 7.03.2006

Ostrzeżenie: (0%)
-----

trzymam loginy i hasla niezakodowane w pliku? nie lepiej zrobic to w bazie i odrazu z kodowaniem md5?
Go to the top of the page
+Quote Post
-Gość_Leon_*-
post
Post #3





Goście
jak bedzie nazwa pliku fdgnfdlsglfdskh4h5gdh5dg4h6s6h46h46hsvb4vcx5n4b5m4gjfg.txt to ktoś odgadnie?
nie chodzi o bezpieczeństwo. Jak rozwiązać problem. Chyba macie gdzieś pod ręką gotowe skrypty to ok.
Go to the top of the page
+Quote Post
suck
post
Post #4





Grupa: Zarejestrowani
Postów: 43
Pomógł: 0
Dołączył: 7.03.2006

Ostrzeżenie: (0%)
-----

wlasnie zdziwilbys sie co ludzie ogladaja, a na bazie danych taki skrypt jest duzo prostrzy do zrobienia
Go to the top of the page
+Quote Post
-Gość_Leon_*-
post
Post #5





Goście
Pomogłeś mi
Go to the top of the page
+Quote Post
suck
post
Post #6





Grupa: Zarejestrowani
Postów: 43
Pomógł: 0
Dołączył: 7.03.2006

Ostrzeżenie: (0%)
-----

tak, bo trzymanie loginow i hasel w pliku na dodatek w zaden sposob niezakodowanych to jest totalna MASAKRA exclamation.gif
Go to the top of the page
+Quote Post
strife
post
Post #7





Grupa: Przyjaciele php.pl
Postów: 2 605
Pomógł: 96
Dołączył: 22.10.2004
Skąd: UK

Ostrzeżenie: (0%)
-----

Witam,

1. Trzymanie takiej bazy w pliku tekstowym nie ma sensu na dłuższą metę ponieważ:
a) Jest to wolniejsze rozwiązanie od baz danych, i posiada mniejsze możliwości.
b) Twój sposób przechowywania login'u uniemożliwia stworzenie użytkownika o nazwie user:tralala:bum
c) Podsumowując nie polecam logowania na plikach tekstowych, no ale nie wiem do czego Ty to wykorzystujesz :)

2. Nawiązując do poprzedniego punktu, proponuje abyś ( jeżeli musisz już to robić na plikach ) przechowywał dane w zserializowanej tablicy ( serialize" title="Zobacz w manualu php" target="_manual ), wtedy będziesz miał łatwiejszy do niej dostęp. Może pokusisz się nawet o stworzenie klasy obsługującej taką małą bazę :>

3. Przechodząc do sedna i odpowiadając już bezpośrednio na Twoje pytanie to:
a) Na początku zwróce uwagę na zapis, jest trochę dziwny, można taki kod zapisać dużo krócej o przedewszystkim czytelniej!
b) Robisz błędy, powinno być method, a nie methot.
c) Poprawiłem ten Twój kod trochę, myślę, że teraz lepiej wygląda :)
[PHP] pobierz, plaintext 
  1. <form name="logowanie" action="?" method="post">
  2. 	Użytkownik: <br />
  3. 	<input type="text" name="user" /> <br />
  4. 	Hasło: <br />
  5. 	<input type="password" name="haslo"><br /><br />
  6. 	<input type="submit" value="Wejdź" />
  7. </form>
  8.  
  9. <?php
  10. function checkPass( $user, $pass )
  11. {  
  12.   $file = file( 'passwords.txt' );
  13.  
  14.   foreach( $file as $k )
  15.   {
  16. 	  $eUser = explode(':', $k );
  17.  
  18. 	  if ( trim( $eUser[ 0 ] ) == $user && trim( $eUser[ 1 ] ) == $pass )	
  19. 	  {
  20. 		  return TRUE;		
  21. 	  }
  22.   }	
  23.  
  24.   return NULL;
  25.  
  26. }
  27. echo '<pre>';
  28. var_export( checkPass( $_POST["user"], $_POST["haslo"]  ) );
  29.  
  30.  
  31.  
  32. ?>
[PHP] pobierz, plaintext


Mój plik z użytkownikami
Kod
strife:demo
demo:demo
admin:demo


Pozdrawiam!

ps. Zobacz może któryś z temat w mojej sygnaturce Ci pomorze ;)


--------------------
Najczęstsze błędy | Jak poprawnie zadać pytanie? | Serwer PHP - od czego zacząć?
SQL Injection | Hashowanie haseł | Jak działają sesje, przykłady

___
The world is yours ...
Go to the top of the page
+Quote Post
-Gość_Leon_*-
post
Post #8





Goście
Nieno dzieki!
Właśnie zauważyłem błąd methot zamiast method. Niewiem jak to sie stało.
A co do tego kodu... to przepisałem słowo w słowo z książki o php... dopiero sie ucze i tak testuje sobie. A do mysql jeszcze nie doszedłem biggrin.gif
Go to the top of the page
+Quote Post
dawhol
post
Post #9





Grupa: Zarejestrowani
Postów: 256
Pomógł: 0
Dołączył: 19.07.2006
Skąd: Ostrów Wielkopolski

Ostrzeżenie: (0%)
-----

ja zaczołem od mysql bo moim zdaniem jest o wiele wiele łatwiejszy od plików dlatego polecam myzqla podstaw nauczysz sie w pare minut smile.gif wiec mysle ze pojdzie ci to szybciej niz z plikami smile.gif


--------------------
Go to the top of the page
+Quote Post
-Gość_Leon_*-
post
Post #10





Goście
Zrobiłem sobie coś do edycji plików.
[PHP] pobierz, plaintext 
  1. <html>
  2. <form method="get" action="boxwrite.php">
  3. <textarea name="memo" rows="30" cols="100">
  4. <?php
  5. if(!$fd = fopen('../komentarze.txt', 'r'))
  6. exit("Przepraszamy, błąd podczas lączenia z bazą danych");
  7. while(!feof($fd)){
  8. $str = fgets($fd);
  9. $str = str_replace("n", "<br>", $str);
  10. echo($str);
  11. }
  12. fclose($fd);
  13. ?>
  14. </textarea><br>
  15. <input type="submit" value="Zapisz">
  16. </form>
  17. </html>
[PHP] pobierz, plaintext

To odczytuje i wysyła dane do zapisu.

Tylko chciałem żeby to było bardziej uniwersalne.
[PHP] pobierz, plaintext 
  1. <html>
  2. <form method="get" action="boxwrite.php">
  3. <textarea name="memo" rows="30" cols="100">
  4. <?php
  5. $file = $_GET["file"];
  6. if(!$fd = fopen($file, 'r'))
  7. exit("Przepraszamy, błąd podczas lączenia z bazą danych");
  8. while(!feof($fd)){
  9. $str = fgets($fd);
  10. $str = str_replace("n", "<br>", $str);
  11. echo($str);
  12. }
  13. fclose($fd);
  14. ?>
  15. </textarea><br>
  16. <input type="submit" value="Zapisz">
  17. </form>
  18. </html>
[PHP] pobierz, plaintext

No i jak wpisuje ......index.php?file=plik.txt to nie działa sad.gif
Co robić?
Go to the top of the page
+Quote Post
rama
post
Post #11





Grupa: Zarejestrowani
Postów: 50
Pomógł: 1
Dołączył: 25.02.2006
Skąd: Trójmiasto

Ostrzeżenie: (0%)
-----

Przejrzałem i zanalizowałem Twój skrypt (bardziej uniwersalny) i powinien działać poprawnie, aczkolwiek występuje w nim jedna drobna różnica w kontraście do poprzedniej wersji, i pewnie stanowi ona przyczynę Twojego problemu, a jest nią odczytywanie zawartości pliku z "lokalnego" folderu, czyli tam gdzie znajduję się Twój skrypt, jednakże poprzednia wersja odczytywała plik z wcześniejszego katalogu (../komentarze.txt).

[PHP] pobierz, plaintext 
  1. <?php
  2. (...)
  3. $file = '../' . $_GET['file'];
  4. if( !$fd = fopen( $file, 'r' ) ) 
  5. {
  6. 	exit('Przepraszamy, błąd podczas lączenia z bazą danych');
  7. }
  8. (...)
  9. ?>
[PHP] pobierz, plaintext

Lub podajesz w url index.php?file=../plik.txt smile.gif
Go to the top of the page
+Quote Post
-Gość_Leon_*-
post
Post #12





Goście
Nie to nie o to chodziło.
Ale rozwiązałem problem.
Zamiast $file wpisałem $plik.
[PHP] pobierz, plaintext 
  1. <?php
  2. $plik = $_GET["file"];
  3. if(!$fd = fopen($plik, 'r'))
  4. exit("Przepraszamy, błąd podczas lączenia z bazą danych");
  5. while(!feof($fd)){
  6. $str = fgets($fd);
  7. $str = str_replace("n", "<br>", $str);
  8. echo($str);
  9. }
  10. fclose($fd);
  11. ?>
[PHP] pobierz, plaintext

I działa!
Go to the top of the page
+Quote Post
« Następny starszy · Przedszkole · Następny nowszy »
 

Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 

Tryb wyświetlania: Standardowy · Przełącz na: Linearny+ · Przełącz na: Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Aktualny czas: 21.08.2025 - 12:48
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn