GitLab: 记录一个循环的功能增强

清疚 2022-12-02 15:20 7阅读 0赞

![在这里插入图片描述][watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center_pic_center_pic_center]  
 GitLab提供了管理用户修改其他人的密码的功能，这给使用GitLab进行功能集成的使用者带来了方便，但是同时该功能确实可能存在隐患和可能引起混乱的地方，于是给此功能添加了一个特性，就是必须修改密码后下次登录必须强制修改密码。于是这引起了很多其他的问题，于是又打开了一个Issue改回去或者说部分改回去，这篇文章来一起看一下这些Issue和对应。

### 目录 ###

*  Issue：13691
 *  Issue：24606
 *  Issue：61587
 *  Issue：36412
 *  Issue：19141
 *  总结
 *  参考文档

--------------------

# Issue：13691 #

*  13691 标题：Allow admin to reset user password and force password reset on next login。标题写的非常清楚，允许admin重设其他用户的密码，被修改了密码的用户下次登录的时候就应该强制密码重设。

首先，作为维护者给出了两种方式，是否应该有个复选框让用户决定，还是直接替用户决定，这就是最佳实践。  
![在这里插入图片描述][watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center]  
被@的回应是：强制重设吧  
![在这里插入图片描述][watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center 1]  
还有一个回应是：always force  
![在这里插入图片描述][2020083120561669.png_pic_center]  
当时还有个路人甲John Stephen赞同了checkbox，被忽视了。  
![在这里插入图片描述][watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center 2]  
然后John Stephen认为，从Web上操作和API操作结果不一致也是一个问题，再起一个Issue吧

# Issue：24606 #

*  24606 标题：Force password reset on next login when change password through API

John Stephen在上述Issue的基础上创建了24606  
![在这里插入图片描述][watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center 3]  
然后正常关闭，就是一个简单的特性增强的示例，如果没有后面的Issue的话。

# Issue：61587 #

标题：Why Force new password after password reset via API。为什么使用Password通过API设定密码之后需要强制密码重设？原因就是前面的Issue的标题和增强的特性的内容，这就是最佳实践吧。  
![在这里插入图片描述][watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center 4]

# Issue：36412 #

标题：Do not force password reset on next login when change password through API。简单来说就是之前的Issue对应的特性删除掉。  
![在这里插入图片描述][watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center 5]

# Issue：19141 #

标题：Do not force password reset on next login when change password through API。类似的Issue还有很多，因为19141的状态为Open，所以其他都关闭了，但是这个三年前开的，现在还没有关闭，三年前刚好24606的Issue关闭了。所基本上从某种程度上来说是一个很具争议性的Issue或者对应方式很具争议性。

实际上确实有很多人都有类似的问题，比如  
![在这里插入图片描述][watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center 6]  
再比如  
![在这里插入图片描述][watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center 7]  
![在这里插入图片描述][watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center 8]  
![在这里插入图片描述][watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center 9]  
然而，状态还是Open。

--------------------

# 总结 #

开源，前向兼容有的时候就会有这个问题，考虑太多，自己很痛苦，考虑太少，用户很痛苦。慢慢来，习惯就好。

# 参考文档 #

https://gitlab.com/gitlab-org/gitlab-foss/-/issues/24606  
https://gitlab.com/gitlab-org/gitlab-foss/-/issues/13691  
https://gitlab.com/gitlab-org/gitlab-foss/-/issues/61587  
https://gitlab.com/gitlab-org/gitlab-foss/-/issues/36412  
https://gitlab.com/gitlab-org/gitlab/-/issues/19141

[watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center_pic_center_pic_center]: /images/20221123/601c08a1408a4b30b42ce241be9e48e6.png
[watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center]: /images/20221123/4d929d0c66644f0dbe51bf3b4dab6b5d.png
[watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center 1]: /images/20221123/181a613882744da68ba59ce9b661b7e2.png
[2020083120561669.png_pic_center]: /images/20221123/157cf472e42c4967ab9941db99191dc0.png
[watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center 2]: /images/20221123/441768162edb4cd6a9bb78dd78e42f76.png
[watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center 3]: /images/20221123/5683e291d6ae45aa82d236f521ab1358.png
[watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center 4]: /images/20221123/1d127918ffe449568dfaae888275501e.png
[watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center 5]: /images/20221123/69c656899dc7447bb5ad6cc9917a0c93.png
[watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center 6]: /images/20221123/527407fd279b4086b8fe57d1179ad79b.png
[watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center 7]: /images/20221123/7c6a176d33b0442391507bdd0a1b3241.png
[watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center 8]: /images/20221123/86b73f10aba640cfbbe79d9fc820822f.png
[watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpdW1pYW9jbg_size_16_color_FFFFFF_t_70_pic_center 9]: /images/20221123/fd93864b98254b909ee52ebabd2e5829.png