mirror of
https://github.com/dani-garcia/vaultwarden.git
synced 2026-04-25 17:25:57 +03:00
[PR #6828] Make available toggling webauthn allow_subdomains #3855
Labels
No labels
SSO
Third party
better for forum
bug
bug
documentation
duplicate
enhancement
future Vault
future Vault
future Vault
good first issue
help wanted
low priority
notes
pull-request
question
troubleshooting
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/vaultwarden#3855
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/dani-garcia/vaultwarden/pull/6828
Author: @Bert-Proesmans
Created: 2/14/2026
Status: 🔄 Open
Base:
main← Head:webauth_origin_proper📝 Commits (1)
19dc021Add option to validate keys where browser reports subdomain of domain URL📊 Changes
2 files changed (+10 additions, -2 deletions)
View changed files
📝
src/api/core/two_factor/webauthn.rs(+7 -2)📝
src/config.rs(+3 -0)📄 Description
This patch makes it possible to have DNS records for subdomains of DOMAIN_URL to point to vaultwarden instances with working webauthn keys.
I verified this code works by running patched vaultwarden instances. This change is foolproof, allow_subdomains might as well have been set to true by default but opted for defensive approach.
Related discussion; https://github.com/dani-garcia/vaultwarden/discussions/6567
Before;
DOMAIN_URL = passwords.my.domain
Before;
DOMAIN_URL = passwords.my.domain
=> webauthn_rs fails on the browser URL passed into auth_requests when doing stuff from hostname zone.passwords.my.domain
After;
DOMAIN_URL = passwords.my.domain
WEBAUTHN_ALLOW_SUBDOMAINS = true
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.