-
-
Couldn't load subscription status.
- Fork 25
Transfer ownership #14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: LazyCat2 <68156188+LazyCat2@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think forcing the user to have 2FA before being able to transfer servers would be a good idea instead of suggesting that any account can transfer a server. 👍🏼
|
The way the 2FA flow works currently means it'll fallback to password auth if no methods are available, if we could use & specify that then this would be perfect. This would protect against situations where someone would somehow get the authentication token, but would be unlikely to have the password for example. |
Signed-off-by: LazyCat2 <68156188+LazyCat2@users.noreply.github.com>
Yeah, me and LazyCat talked about that on the Revolt Development server/original PR a while ago lol. |
|
I don't think there are any substantial blockers that would mean we need to delay the merge here? |
No description provided.