Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: php logowanie
Forum PHP.pl > Forum > PHP
czarkowy
6.08.2017, 12:10:43
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? :_:
viking
6.08.2017, 15:48:47
Sprawdz co zwraca wynik zapytania. Dodatkowo powinieneś bindowac parametry. A hasło przez password_hash.
czarkowy
6.08.2017, 16:09:24
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
viking
6.08.2017, 16:16:45
Z tego wynika ze jest0 rekordów. Zrób var_dump($_POST).
czarkowy
6.08.2017, 16:19:30
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
markuz
6.08.2017, 16:31:17
Masz literówkę, nie mathod tylko method (w formularzu html).
czarkowy
6.08.2017, 16:34:14
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
viking
6.08.2017, 17:08:53
Tak czy inaczej zastosuj http://php.net/manual/en/mysqli-stmt.bind-param.php bo teraz masz otwarty skrypt na ataki.
czarkowy
7.08.2017, 20:23:04
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
viking
7.08.2017, 20:43:41
W dokumentacji to na pewno tak nie wygląda więc masz odpowiedź. Wywal te add_slashes i escape stri ng dodatkowo.
czarkowy
7.08.2017, 21:11:08
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

viking
8.08.2017, 05: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 )
czarkowy
8.08.2017, 10:37:24
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
viking
8.08.2017, 10:54:49
A jak ma wywalić błędy skoro robisz przekierowanie
czarkowy
8.08.2017, 11:11:26
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?
nospor
8.08.2017, 11:17:03
Cytat
no uruchamiam i kurde działa,
Sorki, ze sie wcinam w te interesujaca dyskusje ale skoro dziala to w czym problem?
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.