Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Validate all needed fields for client API login #3251

Merged
merged 1 commit into from
Feb 21, 2023
Merged

Validate all needed fields for client API login #3251

merged 1 commit into from
Feb 21, 2023
+4 −0

Commits on Feb 19, 2023

  1. Validate all needed fields for client API login 

    During the client API login we need to have a `device_identifier`, `device_name` and `device_type`.
When these were not provided Vaultwarden would panic.

This PR add checks for these fields and makes sure it returns a better error message instead of causing a panic.
    @BlackDex
    BlackDex committed Feb 19, 2023
    Configuration menu
    Copy the full SHA
    1ca0d6e View commit details
    Browse the repository at this point in the history