masterking32 / WoWSimpleRegistration

Simple Registration page for TrinityCore/AzerothCore/AshamaneCore/CMangos
https://masterking32.com
GNU General Public License v3.0
254 stars 212 forks source link

Old password is not valid. #80

Open paul-nagorney-icrossing opened 1 year ago

paul-nagorney-icrossing commented 1 year ago

Describe the bug When user tries to change password, they get an error: Old password is not valid.

To Reproduce Steps to reproduce the behavior:

  1. Register an account
  2. User gets the generated password from email, and logs in to the game to ensure its working
  3. Go back to website, click Change Password
  4. Enter username, old password, and new password in the modal
  5. Click Change password on the modal
  6. When the POST happens and page reloads, the user sees the error "Old password is not valid."

Expected behavior Its expected that user's new password is accepted and changed in the game server

Screenshots image

Desktop (please complete the following information):

Additional context I have validated that the POST going from my browser to the server contains the proper values Using google recaptcha v2 $config['server_core'] = 5; $config['srp6_support'] = true; $config['soap_for_register'] = false;

verifySRP6 is returning false on line 388 of user.php image

gmp is enabled in php (verified via phpinfo) image

wsh123321 commented 8 months ago

I also encountered the same problem and found that the verification of SRP6 did not pass. I checked and found that the account verification for both CMANGOS and VMANGOS was the same Here is the difference in verifying passwords, hoping to find a solution.