eryajf / go-ldap-admin

🌉 基于Go+Vue实现的openLDAP后台管理项目
http://ldapdoc.eryajf.net
Apache License 2.0
1.58k stars 265 forks source link

feat:增加设置新密码时强度校验功能 #224

Open liuup opened 1 year ago

liuup commented 1 year ago

在提出此拉取请求时,我确认了以下几点(请复选框):

填写PR内容:

新密码规则:

liuup commented 1 year ago

完善此密码修改功能还需要:

eryajf commented 1 year ago

事实上,如果要严格对密码做校验,还有不少需要考虑的点:

我个人其实对前端不甚精通,所以也有点为难。这里可以作为一个需求征集点,不晓得你前端咋样,如果可以,我们可以在这里详细讨论下功能设计,然后开干。或者其他有前端能力的朋友,也欢迎一起参与这个功能的建设。

liuup commented 1 year ago

事实上,如果要严格对密码做校验,还有不少需要考虑的点:

  • 创建用户的时候,也应该添加校验
  • 同步用户的时候,可能也应该走这个校验,或者不走这个校验,而增加用户首次登陆必须重置密码的功能
  • 如你所言,还应该结合前端页面做配合,这块儿也是一个问题

我个人其实对前端不甚精通,所以也有点为难。这里可以作为一个需求征集点,不晓得你前端咋样,如果可以,我们可以在这里详细讨论下功能设计,然后开干。或者其他有前端能力的朋友,也欢迎一起参与这个功能的建设。

OK明白你的意思,后端确实还需要有另外几个地方也需要加上校验的功能,可能还需要再琢磨一下;然后我对于前端实在一窍不通,前端部分确实不知道该如何修改了QAQ

songlipeng2003 commented 11 months ago

这个为啥不合并进去呢?感觉非常需要这个功能

liuup commented 11 months ago

这个为啥不合并进去呢?感觉非常需要这个功能

你可以看一下上面的回复hhh,我只是push了一个后端校验密码的功能,但是与之配套的前后端对应的UI/提示/测试还需要其他人来一起完成