Skip to content
This repository has been archived by the owner on Mar 9, 2021. It is now read-only.

Add a simple flow to allow changing user passwords. #1122

Closed
2 tasks done
topaz-next-bot opened this issue Dec 17, 2020 · 0 comments
Closed
2 tasks done

Add a simple flow to allow changing user passwords. #1122

topaz-next-bot opened this issue Dec 17, 2020 · 0 comments

Comments

@topaz-next-bot
Copy link

Issue by mrhappyasthma
Sunday Feb 23, 2020 at 04:45:11
Originally opened as: project-topaz/topaz - Issue 379


This is a prerequisite for DarkstarProject/xiloader#7.

This change adds support for a new command in the login code to support
changing a users password. It simply verifies the users login similarly
to the existing flow, and then makes another request over the socket for
the new password. Then it updates the SQL.

Tested locally. Have a set of changes ready for xiloader as well, but
this work is required first.

NOTE: There is definitely some refactoring that could be done in this
file to reduce the amount of nested braces and to reduce duplicated code.
I tried to touch as little existing code as possible in this change to
reduce the risk of introducing a regression.

I affirm:

  • that I agree to Project Topaz's Limited Contributor License Agreement, as written on this date
  • that I've tested my code since the last commit in the PR, and will test after any later commits
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant