Skip to content

Dummy Identifier value to skip screen - #25

Open
Timshel wants to merge 1 commit into
vaultwarden:v2026.6.2from
vw-contrib:vaultwarden
Open

Dummy Identifier value to skip screen#25
Timshel wants to merge 1 commit into
vaultwarden:v2026.6.2from
vw-contrib:vaultwarden

Conversation

@Timshel

@Timshel Timshel commented May 21, 2026

Copy link
Copy Markdown

With the recent change around /organizations/domain/sso/verified it now always return the hardcoded value.

I think it makes sense to set it in the client, this allows to use a cleaner /#/sso for those who prefer to skip the email prompt (logout still redirect to login form with email sadly). And if it needs to change again lessen the risks of someone using an outdated value.

If called with the hardcoded identifier auto-enroll-status can return a different Organization.
I believe it does not cause issues but at the same time the value is later used in

@Timshel
Timshel changed the base branch from v2026.4.1 to v2026.6.2 June 25, 2026 04:50
@Timshel

Timshel commented Jun 26, 2026

Copy link
Copy Markdown
Author

@stefan0xC Hey,

Recently started building the Firefox extension because I got tired of putting a dummy email and then pressing enter, then backtracking since I want SSO and not password login.

Wanted to propose it to the SSO user of Vaultwarden too, but without this patch the extension drop you in the identifier screen. Might be able to set the identifier, but I believe this patch is relatively simple and make sense since it's not really needed for the user to have to set and identifier.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant