Skip to content

修复事务实效,并新增rember me功能#222

Open
zhaieryuan wants to merge 4 commits intoiflytek:mainfrom
zhaieryuan:pr/20260402-bug-fix
Open

修复事务实效,并新增rember me功能#222
zhaieryuan wants to merge 4 commits intoiflytek:mainfrom
zhaieryuan:pr/20260402-bug-fix

Conversation

@zhaieryuan
Copy link
Copy Markdown
Contributor

Summary

  • 1、给登陆页面添加 rember me功能,能够简单快速的登陆
  • 2、修复仓库中登陆过程,spring 事务实效,导致记录用户登陆账号密码错误次数,功能实效
image

Validation

  • [ y] Backend tests passed
  • [ y] Frontend typecheck/build passed
  • [ y] OpenAPI SDK regenerated or checked when API contracts changed
  • [ y] Smoke test run when relevant

zhaieryuan added 3 commits April 2, 2026 10:35
* main:
  docs: add Kubernetes deployment guide with overlays structure (iflytek#219)
  docs: simplify runtime script usage (iflytek#217)
  fix: add --public-url parameter for docker deployment (iflytek#216)
  chore: update .gitignore to include .codex directory (iflytek#211)
  organize .gitignore (iflytek#206)
  docs: add VitePress bilingual documentation site (iflytek#208)
@wowo-zZ
Copy link
Copy Markdown
Collaborator

wowo-zZ commented Apr 3, 2026

关注一下后端单测哈。

@wowo-zZ
Copy link
Copy Markdown
Collaborator

wowo-zZ commented Apr 3, 2026

另外看了具体的改动,感谢你把登录事务和 Swagger 访问问题一起补上。
不过这个 PR 合并前还需要调整两点:

  1. remember me 不能把明文密码存进 localStorage,建议改成只记用户名,或者后续单独做服务端 remember-me token。
  2. LocalAuthFailedService 的拆分思路没问题,但现在字段注入把 LocalAuthServiceTest 打挂了,建议改成构造器注入并补测试。

@zhaieryuan
Copy link
Copy Markdown
Contributor Author

嗯,看到了 workflow ,正在调整

…构造注入, 并补充单元测试, fix LocalAuthServiceTest 打挂 的问题
@zhaieryuan zhaieryuan closed this Apr 3, 2026
@zhaieryuan
Copy link
Copy Markdown
Contributor Author

重新调整以下功能:
1、 remember me 不能把明文密码存进 localStorage,
2、修改 LocalAuthFailedService 构造注入,
3、补充Auth failed 单元测

@zhaieryuan zhaieryuan reopened this Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants