Błąd z formularzem rejestracji (hashowanie hasła)

[TheTroy]

Witam. Otóż mam problem z formularzem rejestracji wywala mi taki oto komunikat:

Cytat

Strict standards: Non-static method user::passSalter() should not be called statically in

C:\wamp64\www\register.php on line 75

oto kod z 75 linii:

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

dane które są pobierane z formularza:

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

a tutaj jest sam formularz HTML:

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

Wie może ktoś jakby naprawić ten błąd?

[TheTroy]

8 godzin temu, maxaz napisał:

spróbuj:

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

Jak chcesz samo hash, to po prostu używasz go na stringu:

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

 

Niestety dalej nie działa ;/ może podam ci cały kod rejestracji i go se ogarniesz:

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

a passSalter pobiera z innego pliku tu poniżej masz zawartość jak to "passSalter" wygląda w 2 pliku:

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

Tu powyżej gdzie pisze:

[email protected]

powinno być: $ @ @ # $ # @ $ dodałem odstęp od tego aby nie wywalało tego tekstu bo wcięło się to co wyżej .

[maxaz]

Używanie md5 do hashowania haseł w 2018 nie jest zbyt dobrym pomysłem. Ponownie namówię Cię do skorzystania z wbudowanej w PHP funkcji hash -- jest ona bezpieczniejsza. Przy sprawdzaniu hasła, po prostu porównujesz hash z inputa z hashem w bazie danych.

[TheTroy]

18 godzin temu, Aranthor napisał:

I ponownie rozchodzi się o nieczytanie treści błędów. Przecież masz napisane:

Podczas dodawania nowego użytkownika (INSERT INTO users) musisz podać wartość kolumny waluta.

Ooo no zapomniałem że dodałem dodatkowe wartości w bazie danych dzięki mordeczko pomogłeś!

Już wszystko działa

Temat do zamknięcia.

[TheTroy]

6 godzin temu, maxaz napisał:

Używanie md5 do hashowania haseł w 2018 nie jest zbyt dobrym pomysłem. Ponownie namówię Cię do skorzystania z wbudowanej w PHP funkcji hash -- jest ona bezpieczniejsza. Przy sprawdzaniu hasła, po prostu porównujesz hash z inputa z hashem w bazie danych.

Hmm no okey to w takim razie jak mam to przerobić żeby działało bo gdy używam teraz:

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

zamiast:

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

to już nie wyświetla żadnych komunikatów z błędami, ale wyskakuje komunikat z echo że nie można było się zarejestrować. No i niestety nie działa ;/

[Aranthor]

23 godziny temu, maxaz napisał:

Jak chcesz samo hash, to po prostu używasz go na stringu:

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

Funkcja hash przyjmuje 2 argumenty: rodzaj algorytmu i dane do przetworzenia.

[TheTroy]

Teraz gdy zastosowałem to:

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

wywala mi taki błąd:

Cytat

Warning: hash() expects at least 2 parameters, 1 given in 

dalszy kod wygląda tak: 

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

i próbowałem nawet zmieniać w VALUES $pass na $hashedPassword

kombinowałem z tym ale nadal ten sam błąd mi pokazuje ;/

Może to przez to że korzystam z wampp'a a wcześniej korzystałem z xampp'a i tam wszystko mi działało dobrze, może to przez to? bo nawet bez hash'owania hasła już nie wyskakuje mi ten Warning, tylko już sama wiadomość:

Cytat

Wystąpił błąd w zapytaniu i nie udało się zarejestrować użytkownika.

z ten linii:

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

 

[Aranthor]

14 minut temu, TheTroy napisał:

wywala mi taki błąd:

Cytat

Treść błędu mówi sama za siebie, wystarczyło go przeczytać. Zresztą napisałem o tym 2 posty wyżej: funkcja hash potrzebuje dwóch argumentów:

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

 

Dobrze by było wyświetlić ostatni błąd MySQL, aby dowiedzieć się, dlaczego zapytanie nie zadziałało:

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

 

[maxaz]

Metoda passSalter nie jest statyczna — aby jej użyć, musisz utworzyć instancję klasy user.

Gdybyś wstawił kod klasy user, byłoby dużo łatwiej.

Poza tym, funkcja 

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

 nie jest wystarczająca do zabezpieczenia hasła?

[TheTroy]

1 godzinę temu, maxaz napisał:

Metoda passSalter nie jest statyczna — aby jej użyć, musisz utworzyć instancję klasy user.

Gdybyś wstawił kod klasy user, byłoby dużo łatwiej.

Poza tym, funkcja 

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

 nie jest wystarczająca do zabezpieczenia hasła?

Mógłbyś mi w tym pomóc bo za bardzo nie wiem jak to zrobić zamiast tego:

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

czy też po user:: to zastosować?

Jakbyś mi z tym pomógł byłbym bardzo ci wdzięczny.

[TheTroy]

5 godzin temu, Aranthor napisał:

Treść błędu mówi sama za siebie, wystarczyło go przeczytać. Zresztą napisałem o tym 2 posty wyżej: funkcja hash potrzebuje dwóch argumentów:

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

 

Dobrze by było wyświetlić ostatni błąd MySQL, aby dowiedzieć się, dlaczego zapytanie nie zadziałało:

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

 

Okey tera wywala taki oto błąd:

Cytat

Wystąpił błąd w zapytaniu i nie udało się zarejestrować użytkownika: Field 'waluta' doesn't have a default value

może to jest związane gdzieś z tym kodem? 

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

Ten powyższy kod jest inplementowany do rejestracji:

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

Może to ci w czymś pomoże.

[maxaz]

spróbuj:

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

Jak chcesz samo hash, to po prostu używasz go na stringu:

To jest ukryta treść, proszę

• Zaloguj się
• lub
• Zarejestruj się

 

[Aranthor]

I ponownie rozchodzi się o nieczytanie treści błędów. Przecież masz napisane:

Cytat

Field 'waluta' doesn't have a default value

Podczas dodawania nowego użytkownika (INSERT INTO users) musisz podać wartość kolumny waluta.