Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
 Reply to this topicStart new topic
> php logowanie
czarkowy
post 6.08.2017, 12:10:43
Post #1





Grupa: Zarejestrowani
Postów: 31
Pomógł: 1
Dołączył: 6.08.2017
Skąd: sdsadada

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

Siemka,
zrobiłem logowanie w php i jest mały problem, otóż za każdym raze jest niepoprawne logowanie, nie ważne czy dobry login i hasło czy zły

index.php
[HTML] pobierz, plaintext 
  1. <!DOCTYPE html>
  2. <html lang="pl">
  3. <head>
  4. 	<meta charset="UTF-8">
  5. 	<title>Home Page</title>
  6. 	<link rel="stylesheet" href="index.css">
  7. 	<link rel="stylesheet" href="register.css">
  8. </head>
  9. <body>
  10.  
  11. <form class="form-contener" action="signin.php" mathod="POST">
  12. 	<input type="text" class="inputText" name="username" placeholder="Your username">
  13. 	<input type="password" class="inputText" name="pass" placeholder="Your password">
  14. 	<button type="submit" class="inputSubmit" name="login">Sign In</button>
  15. </form>
  16.  
  17. </body>
  18. </html>
[HTML] pobierz, plaintext


signin.php
[PHP] pobierz, plaintext 
  1. <?php 
  2.  
  3. include 'db_connect.php';
  4.  
  5. $username = isset($_POST['username']) ? $_POST['username'] : "";
  6. $pass = isset($_POST['pass']) ? $_POST['pass'] : "";
  7.  
  8. $sql = "SELECT * FROM users WHERE username = '$username' AND pass = '$pass'";
  9.  
  10. $result = mysqli_query($conn, $sql);
  11.  
  12. if(!$row = mysqli_fetch_assoc($result)) {
  13. 	echo "Niezalogowany";
  14. } else {
  15. 	echo "Zalogowany";
  16. }
[PHP] pobierz, plaintext


o co moze chodzic? :_:
Go to the top of the page
+Quote Post
viking
post 6.08.2017, 15:48:47
Post #2





Grupa: Zarejestrowani
Postów: 6 365
Pomógł: 1114
Dołączył: 30.08.2006

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

Sprawdz co zwraca wynik zapytania. Dodatkowo powinieneś bindowac parametry. A hasło przez password_hash.


--------------------
Odpowiedzi na często zadawane pytania:
Konfiguracja serwera Apache + PHP 7 pod Windows | Kodowanie znaków na stronach www | PHPTAL w Zend Framework | Programowanie obiektowe w PHP | PDO uniwersalnym sposobem na obsługę baz danych | Kurs PHP (część 1): Podstawowy opis języka
Go to the top of the page
+Quote Post
czarkowy
post 6.08.2017, 16:09:24
Post #3





Grupa: Zarejestrowani
Postów: 31
Pomógł: 1
Dołączył: 6.08.2017
Skąd: sdsadada

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

Cytat(viking @ 6.08.2017, 16:48:47 ) *
Sprawdz co zwraca wynik zapytania. Dodatkowo powinieneś bindowac parametry. A hasło przez password_hash.


bindowac parametry tzn?

i z hashowaniem to wiem, ale po co to teraz robić jak nic nawet nie dziala ;p

dodałem var_dump($result);
i zwraca mi to:
[PHP] pobierz, plaintext 
  1. E:\Programy\wamp64\www\testproject\signin.php:17:
  2. object(mysqli_result)[2]
  3.   public 'current_field' => int 0
  4.   public 'field_count' => int 5
  5.   public 'lengths' => null
  6.   public 'num_rows' => int 0
  7.   public 'type' => int 0
[PHP] pobierz, plaintext


w ogóle nie wiem czy to jakoś źle dane nie sa przesylane ;p
bo var_dump($username) i var_dump($pass) zwracają mi puste stringi

@edit

$username = isset($_POST['username']) ? $_POST['username'] : "";
$pass = isset($_POST['pass']) ? $_POST['pass'] : "";

tutaj chyba w ogóle username i pass nie sa rozpoznawane i po prostu program wybiera ostatnia opcje czyli pustego stringa ;x bo jak tam wpisalem poprawne dane to mozna sie zalogowac


Ten post edytował czarkowy 6.08.2017, 16:17:43
Go to the top of the page
+Quote Post
viking
post 6.08.2017, 16:16:45
Post #4





Grupa: Zarejestrowani
Postów: 6 365
Pomógł: 1114
Dołączył: 30.08.2006

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

Z tego wynika ze jest0 rekordów. Zrób var_dump($_POST).


--------------------
Odpowiedzi na często zadawane pytania:
Konfiguracja serwera Apache + PHP 7 pod Windows | Kodowanie znaków na stronach www | PHPTAL w Zend Framework | Programowanie obiektowe w PHP | PDO uniwersalnym sposobem na obsługę baz danych | Kurs PHP (część 1): Podstawowy opis języka
Go to the top of the page
+Quote Post
czarkowy
post 6.08.2017, 16:19:30
Post #5





Grupa: Zarejestrowani
Postów: 31
Pomógł: 1
Dołączył: 6.08.2017
Skąd: sdsadada

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

Cytat(viking @ 6.08.2017, 17:16:45 ) *
Z tego wynika ze jest0 rekordów. Zrób var_dump($_POST).


var_dump($_POST) zwraca:
[PHP] pobierz, plaintext 
  1. E:\Programy\wamp64\www\testproject\signin.php:19:
  2. array (size=0)
  3.   empty
[PHP] pobierz, plaintext
Go to the top of the page
+Quote Post
markuz
post 6.08.2017, 16:31:17
Post #6





Grupa: Zarejestrowani
Postów: 1 240
Pomógł: 278
Dołączył: 11.03.2008

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

Masz literówkę, nie mathod tylko method (w formularzu html).


--------------------
Blog
Go to the top of the page
+Quote Post
czarkowy
post 6.08.2017, 16:34:14
Post #7





Grupa: Zarejestrowani
Postów: 31
Pomógł: 1
Dołączył: 6.08.2017
Skąd: sdsadada

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

Cytat(markuz @ 6.08.2017, 17:31:17 ) *
Masz literówkę, nie mathod tylko method (w formularzu html).


...od 2 dni z tym się głowie i walcze...a tu takie małe gówienko..uhh
dzięki wielkie smile.gif
Go to the top of the page
+Quote Post
viking
post 6.08.2017, 17:08:53
Post #8





Grupa: Zarejestrowani
Postów: 6 365
Pomógł: 1114
Dołączył: 30.08.2006

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

Tak czy inaczej zastosuj http://php.net/manual/en/mysqli-stmt.bind-param.php bo teraz masz otwarty skrypt na ataki.


--------------------
Odpowiedzi na często zadawane pytania:
Konfiguracja serwera Apache + PHP 7 pod Windows | Kodowanie znaków na stronach www | PHPTAL w Zend Framework | Programowanie obiektowe w PHP | PDO uniwersalnym sposobem na obsługę baz danych | Kurs PHP (część 1): Podstawowy opis języka
Go to the top of the page
+Quote Post
czarkowy
post 7.08.2017, 20:23:04
Post #9





Grupa: Zarejestrowani
Postów: 31
Pomógł: 1
Dołączył: 6.08.2017
Skąd: sdsadada

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

Cytat(viking @ 6.08.2017, 18:08:53 ) *
Tak czy inaczej zastosuj http://php.net/manual/en/mysqli-stmt.bind-param.php bo teraz masz otwarty skrypt na ataki.


tak to powinno wyglądać? :

[PHP] pobierz, plaintext 
  1. <?php
  2. session_start();
  3.  
  4. if(isset($_SESSION['name'])){
  5. 	header('Location: index.php');
  6. }
  7.  
  8. include 'db_connect.php';
  9.  
  10. $firstname = isset($_POST['firstname']) ? $_POST['firstname'] : "";
  11. $username = isset($_POST['username']) ? $_POST['username'] : "";
  12. $email = isset($_POST['email']) ? $_POST['email'] : "";
  13. $pass = isset($_POST['pass']) ? $_POST['pass'] : "";
  14. $pass2 = isset($_POST['pass2']) ? $_POST['pass2'] : "";
  15.  
  16. if($pass == $pass2){
  17.  
  18. 	$username = addslashes($username);
  19. 	$pass = addslashes($pass);
  20.  
  21. 	$username = mysqli_real_escape_string($conn, $username);
  22. 	$pass = mysqli_real_escape_string($conn, $pass);
  23.  
  24. 	$pass = hash('sha256', '<>?/!@#$%^&*()-=+:'.$pass);
  25.  
  26. 	$stmt = mysqli_prepare($conn, "INSERT INTO users VALUES(?, ?, ?, ?)");
  27. 	mysqli_stmt_bind_param($stmt, $firstname, $username, $email, $pass);
  28.  
  29. 	/*$sql = "INSERT INTO users (firstname, username, email, pass) VALUES ('$firstname', '$username', '$email', '$pass')";
  30. 	$result = $conn->query($sql);
  31. 	*/
  32. 	mysqli_stmt_execute($stmt);
  33.  
  34. 	$_SESSION['name'] = 'session: '.$username;
  35. 	$_SESSION['username'] = $username;
  36.  
  37. 	header("Location: index.php");
  38. }
  39.  
  40. mysqli_stmt_close($stmt);
  41. mysqli_close($conn);
  42.  
[PHP] pobierz, plaintext


Ten post edytował czarkowy 7.08.2017, 20:34:14
Go to the top of the page
+Quote Post
viking
post 7.08.2017, 20:43:41
Post #10





Grupa: Zarejestrowani
Postów: 6 365
Pomógł: 1114
Dołączył: 30.08.2006

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

W dokumentacji to na pewno tak nie wygląda więc masz odpowiedź. Wywal te add_slashes i escape stri ng dodatkowo.


--------------------
Odpowiedzi na często zadawane pytania:
Konfiguracja serwera Apache + PHP 7 pod Windows | Kodowanie znaków na stronach www | PHPTAL w Zend Framework | Programowanie obiektowe w PHP | PDO uniwersalnym sposobem na obsługę baz danych | Kurs PHP (część 1): Podstawowy opis języka
Go to the top of the page
+Quote Post
czarkowy
post 7.08.2017, 21:11:08
Post #11





Grupa: Zarejestrowani
Postów: 31
Pomógł: 1
Dołączył: 6.08.2017
Skąd: sdsadada

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

Cytat(viking @ 7.08.2017, 21:43:41 ) *
W dokumentacji to na pewno tak nie wygląda więc masz odpowiedź. Wywal te add_slashes i escape stri ng dodatkowo.


dziwne to jest ;p

takie coś? :

[PHP] pobierz, plaintext 
  1. <?php
  2. session_start();
  3.  
  4. if(isset($_SESSION['name'])){
  5. 	header('Location: index.php');
  6. }
  7.  
  8. include 'db_connect.php';
  9.  
  10. $firstname = isset($_POST['firstname']) ? $_POST['firstname'] : "";
  11. $username = isset($_POST['username']) ? $_POST['username'] : "";
  12. $email = isset($_POST['email']) ? $_POST['email'] : "";
  13. $pass = isset($_POST['pass']) ? $_POST['pass'] : "";
  14. $pass2 = isset($_POST['pass2']) ? $_POST['pass2'] : "";
  15.  
  16. if($pass == $pass2){
  17.  
  18. 	$pass = hash('sha256', '<>?/!@#$%^&*()-=+:'.$pass);
  19.  
  20. 	$stmt = mysqli_prepare($conn, "INSERT INTO users VALUES (?, ?, ?, ?)");
  21. 	mysqli_stmt_bind_param($stmt, 'ss', $firstname_stmt, $username_stmt, $email_stmt, $pass_stmt);
  22.  
  23. 	$firstname_stmt = $firstname;
  24. 	$username_stmt = $username;
  25. 	$email_stmt = $email;
  26. 	$pass_stmt = $pass;
  27.  
  28. 	mysqli_stmt_execute();
  29.  
  30. 	/*$sql = "INSERT INTO users (firstname, username, email, pass) VALUES ('$firstname', '$username', '$email', '$pass')";
  31. 	$result = $conn->query($sql);
  32. 	*/
  33. 	$_SESSION['name'] = 'session: '.$username_stmt;
  34. 	$_SESSION['username'] = $username_stmt;
  35.  
  36. 	header("Location: index.php");
  37. }
  38.  
  39. mysqli_stmt_close($stmt);
  40. mysqli_close($conn);
  41.  
[PHP] pobierz, plaintext

Go to the top of the page
+Quote Post
viking
post 8.08.2017, 05:42:02
Post #12





Grupa: Zarejestrowani
Postów: 6 365
Pomógł: 1114
Dołączył: 30.08.2006

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

Uruchomiłeś ten kod? Z dokumentacji: A string that contains one or more characters which specify the types for the corresponding bind variables. Ile masz bind values? Ja tam widzę 4. Najpierw podstawiasz zmienne, potem je tworzysz.
Definicja: bool mysqli_stmt_execute ( mysqli_stmt $stmt )

Ten post edytował viking 8.08.2017, 06:51:38


--------------------
Odpowiedzi na często zadawane pytania:
Konfiguracja serwera Apache + PHP 7 pod Windows | Kodowanie znaków na stronach www | PHPTAL w Zend Framework | Programowanie obiektowe w PHP | PDO uniwersalnym sposobem na obsługę baz danych | Kurs PHP (część 1): Podstawowy opis języka
Go to the top of the page
+Quote Post
czarkowy
post 8.08.2017, 10:37:24
Post #13





Grupa: Zarejestrowani
Postów: 31
Pomógł: 1
Dołączył: 6.08.2017
Skąd: sdsadada

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

Cytat(viking @ 8.08.2017, 06:42:02 ) *
Uruchomiłeś ten kod? Z dokumentacji: A string that contains one or more characters which specify the types for the corresponding bind variables. Ile masz bind values? Ja tam widzę 4. Najpierw podstawiasz zmienne, potem je tworzysz.
Definicja: bool mysqli_stmt_execute ( mysqli_stmt $stmt )


no uruchamiam i kurde działa, wlasnie gdyby jakies bledy wyskakiwaly to łatwiej by to szlo ;p

teraz takie coś mam:

[PHP] pobierz, plaintext 
  1. <?php
  2. session_start();
  3.  
  4. if(isset($_SESSION['name'])){
  5. 	header('Location: index.php');
  6. }
  7.  
  8. include 'db_connect.php';
  9.  
  10. 	$stmt = mysqli_prepare($conn, "INSERT INTO users(firstname, username, email, pass) VALUES (?, ?, ?, ?)");
  11. 	mysqli_stmt_bind_param($stmt, 'ssss', $firstname_stmt, $username_stmt, $email_stmt, $pass_stmt);
  12.  
  13. 	$firstname_stmt = isset($_POST['firstname']) ? $_POST['firstname'] : "";
  14. 	$username_stmt = isset($_POST['username']) ? $_POST['username'] : "";
  15. 	$email_stmt = isset($_POST['email']) ? $_POST['email'] : "";
  16. 	$pass_stmt = isset($_POST['pass']) ? $_POST['pass'] : "";
  17. 	$pass2 = isset($_POST['pass2']) ? $_POST['pass2'] : ""; 
  18.  
  19. 	$pass_stmt = hash('sha256', '<>?/!@#$%^&*()-=+:'.$pass_stmt);
  20.  
  21. 	mysqli_stmt_execute($stmt);
  22.  
  23. 	$_SESSION['name'] = 'session: '.$username_stmt;
  24. 	$_SESSION['username'] = $username_stmt;
  25.  
  26. 	header("Location: index.php");
  27.  
  28. mysqli_stmt_close($stmt);
  29. mysqli_close($conn);
  30.  
[PHP] pobierz, plaintext
Go to the top of the page
+Quote Post
viking
post 8.08.2017, 10:54:49
Post #14





Grupa: Zarejestrowani
Postów: 6 365
Pomógł: 1114
Dołączył: 30.08.2006

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

A jak ma wywalić błędy skoro robisz przekierowanie


--------------------
Odpowiedzi na często zadawane pytania:
Konfiguracja serwera Apache + PHP 7 pod Windows | Kodowanie znaków na stronach www | PHPTAL w Zend Framework | Programowanie obiektowe w PHP | PDO uniwersalnym sposobem na obsługę baz danych | Kurs PHP (część 1): Podstawowy opis języka
Go to the top of the page
+Quote Post
czarkowy
post 8.08.2017, 11:11:26
Post #15





Grupa: Zarejestrowani
Postów: 31
Pomógł: 1
Dołączył: 6.08.2017
Skąd: sdsadada

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

Cytat(viking @ 8.08.2017, 11:54:49 ) *
A jak ma wywalić błędy skoro robisz przekierowanie


wywalilem i tez nie ma zadnych

to wyzej jest okej czy nie?
Go to the top of the page
+Quote Post
nospor
post 8.08.2017, 11:17:03
Post #16





Grupa: Moderatorzy
Postów: 36 455
Pomógł: 6292
Dołączył: 27.12.2004
Cytat
no uruchamiam i kurde działa,
Sorki, ze sie wcinam w te interesujaca dyskusje ale skoro dziala to w czym problem?


--------------------

"Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista
"Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer
Go to the top of the page
+Quote Post
« Następny starszy · PHP · 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 Wersja Lo-Fi Aktualny czas: 26.04.2024 - 08:00
Powered By IP.Board © 2024  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn