Re:Vaultwarden commit introduces SSO using OpenID Connect
by sandeepkd on Friday August 15, 2025 @12:24PM
(#44914308)
Interesting to see a PR being merged after good 2 years. Thought about the idea of reviewing the changes for self learning, however the number of files involved made me to give up on that idea soon enough. 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. Unless the code review is managed internally, the PR gives an impression of mostly happy paths.
[ Parent
| on HN ]
There are no comments on this comment yet.