Skip to content

Rollback password validation feature on registration page

What does this MR do and why?

  • ::License.feature_available?(:password_complexity)true 时,密码复杂度前端 js 正常工作
  • ::License.feature_available?(:password_complexity)false 时,密码复杂度前端 js 会抛错,导致后续js无法运行
    • 若此时 new PasswordValidator() 为页面最后一行 js 则不会影响页面 feature
    • 若此时 new PasswordValidator() 后续还有其他 feature 逻辑, 则会导致其他 feature 在前端 breakdown

对于这个问题,已经提出upstream MR 来 fix: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/92746, 在它merge之前:

  • 对于极狐
    • 目前 极狐SaaS 15.2 中的 ::License.feature_available?(:password_complexity) 为 true,不会受影响
    • 极狐Self Manage 15.2中 对于 JH starter 即 ::License.feature_available?(:password_complexity) 为 false 的用户,因为没有短信验证这个功能,抛错不会影响页面 feature
  • 对于 upstream 影响相同

为了预防可能的其他风险,在 https://gitlab.com/gitlab-org/gitlab/-/merge_requests/92746 merge 之前,决定 rollback 注册页密码复杂度相关代码,再发布15.2, merge之后,前端抛错消除之后,再恢复相关代码。

Screenshots or screen recordings

How to set up and validate locally

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Kun Qian 编辑于

合并请求报告

加载中