Skip to content
This repository has been archived by the owner on Oct 17, 2023. It is now read-only.

version 1 #7

Merged
merged 7 commits into from
Oct 14, 2023
Merged

version 1 #7

merged 7 commits into from
Oct 14, 2023

Conversation

whxxxxxxxxxx
Copy link
Contributor

实现基本增删改查功能和数据库存储

@whxxxxxxxxxx
Copy link
Contributor Author

调整了项目中go.mod 和 go.sum的位置 保证项目上传通过checks

@NX-Official
Copy link
Member

看着已经很不错了 ❤️

@Esonhugh
Copy link

Esonhugh commented Oct 7, 2023

  1. ParseToken 可以考虑用 Context 传递 Jwt 里的信息到后面的组件 而不是二次 ParseToken 取出 Claims
  2. GetOne Update Delete 这里直接使用了 传递的 Tid 字符串 注意下 越权查看 更新 删除 ,认证用户不一定对任意 Task 做处理的权限

@whxxxxxxxxxx
Copy link
Contributor Author

基础功能已经基于框架实现,但是项目还存在如下需要改进的部分,受限于个人能力,需要后续改进:

  1. 报错最好统一到一个包中,对于各种错误直接调用包;
  2. 对象的实例化处理可以使用sync.once;
  3. 对于身份的校验可以直接加在中间件中,从中间件通过context传递信息;
  4. 由于token24小时才会失效,当用户密码变更时还可以使用原token离线访问,存在安全隐患;
  5. 需要添加日志功能;
  6. 可以利用redis处理缓存。

@NX-Official NX-Official merged commit 6eb052b into hduhelp:main Oct 14, 2023
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants