Hacker News 中文摘要

RSS订阅

Vaultwarden提交引入OpenID Connect实现SSO -- Vaultwarden commit introduces SSO using OpenID Connect

文章摘要

Timshel在GitHub上提交了一个关于在Vaultwarden中实现OpenID Connect单点登录(SSO)的拉取请求,旨在增强用户身份验证的安全性和便捷性。

文章总结

文章主要内容:

在GitHub上的Vaultwarden项目中,Timshel提交了一个名为“SSO using OpenID Connect”的Pull Request(#3899),旨在为Vaultwarden添加对OpenID Connect的支持,以实现通过外部SSO(单点登录)进行身份验证的功能。该功能允许用户通过SSO控制谁可以使用Vault,而无需依赖邀请或LDAP。尽管SSO用于身份验证,但用户仍然需要主密码来解密数据,且该密码不由SSO控制。

该PR基于之前的多个PR(#2787、#2449、#3154),并得到了多位贡献者的帮助。Timshel在PR中提供了详细的文档和测试说明,特别是针对Keycloak的测试环境。此外,他还提供了两个前端修改版本,一个与现有版本兼容,另一个默认将用户重定向到SSO页面。

尽管该功能已经经过测试并准备就绪,但项目维护者BlackDex表示由于时间有限,且存在多个SSO相关的PR,他对合并该PR持谨慎态度。他建议创建一个半支持的发布分支来包含SSO功能,但Timshel认为这可能会导致数据库迁移问题,并建议直接合并该PR。

最终,项目所有者dani-garcia在2025年8月8日批准并合并了该PR,SSO功能将首先在vaultwarden/server:testing镜像中提供,预计在2-4周内发布稳定版本。

关键点: - 该PR为Vaultwarden添加了OpenID Connect支持,允许通过外部SSO进行身份验证。 - 主密码仍然需要,且不由SSO控制。 - 该PR基于之前的多个PR,并得到了多位贡献者的帮助。 - 维护者BlackDex对合并持谨慎态度,建议创建半支持的分支,但Timshel认为直接合并更为合适。 - 最终,dani-garcia批准并合并了该PR,SSO功能将在测试镜像中提供,并计划在未来发布稳定版本。

评论总结

评论内容总结:

  1. 功能适用性:评论1(DoctorOW)指出,该功能似乎更适用于企业/商业环境,并询问Vaultwarden是否有商业用户。作者提到,在其工作经历中,使用SSO的大型公司通常对自托管开源工具持谨慎态度。

    • 引用1:"Most of the comments seem to confirm that this feature is more intended for corporate/business environments."
    • 引用2:"Everywhere I've worked that was big enough to use SSO was also wary of selfhosting FOSS tools."
  2. 代码审查质量:评论2(sandeepkd)提到,尽管PR合并前有大量评论(610条),但这些评论主要围绕主题而非代码变更。作者认为,除非代码审查在内部进行,否则PR可能只关注了“happy paths”。

    • 引用1:"The number of comments (610) gave an impression that the PR must have been reviewed thoroughly, however a close look tells that the comments are mostly about the topic itself, not about the code changes."
    • 引用2:"Unless the code review is managed internally, the PR gives an impression of mostly happy paths."
  3. 安全性问题:评论3(xxmarkuski)直接询问Vaultwarden的安全性,但没有提供具体背景或细节。

    • 引用1:"How secure is Vaultwarden?"

总结:评论主要围绕Vaultwarden的功能适用性、代码审查质量和安全性展开,观点较为分散,缺乏深入讨论。