MaaAssistantArknights / MaaBackendCenter

MAA Copilot Backend
https://prts.plus
GNU Affero General Public License v3.0
73 stars 50 forks source link

修复登录不校验邮箱是否激活的问题,修复注册不发送激活邮件的问题,修复可能的多线程数据争用问题 #137

Closed Lixuhuilll closed 1 year ago

Lixuhuilll commented 1 year ago

95 #98

Lixuhuilll commented 1 year ago

顺带移除了多余的请求发送激活码的接口

Lixuhuilll commented 1 year ago

排查GIT历史后,发现是本次提交导致的问题:https://github.com/MaaAssistantArknights/MaaBackendCenter/commit/6dd04aec07231338e6e05a42a35dd054a77de5ae 想知道这次修改的目的是什么,不知道我本次的修复是否会导致之前想实现的功能没达成

Lixuhuilll commented 1 year ago

算是看明白了,他想让前端先请求验证码,注册时要携带验证码,这样收不到验证码的邮箱就根本不会注册进来。结果他先是没判断验证码为Null的情况,前端也没和他做协调适配,导致最终注册变成不需要验证邮箱的了。。。。

Lixuhuilll commented 1 year ago

如果是希望不保存那么多未激活账号,可以找个时间段删一遍。维持原有思路也行,要判定好 Null、限制请求次数,然后要和前端交涉。

Lixuhuilll commented 1 year ago

感觉还是他原有的想法好一些,已入库的未激活用户再清除感觉不太完美。现在注册时必须先获取验证码,然后发起注册请求,验证码匹配后才能注册成功。并且限制了验证码发送接口的请求次数。