1ca0d6e Validate all needed fields for client API login
📊 Changes
1 file changed (+4 additions, -0 deletions)
View changed files
📝src/api/identity.rs (+4 -0)
📄 Description
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.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.
## 📋 Pull Request Information
**Original PR:** https://github.com/dani-garcia/vaultwarden/pull/3251
**Author:** [@BlackDex](https://github.com/BlackDex)
**Created:** 2/16/2023
**Status:** ✅ Merged
**Merged:** 2/21/2023
**Merged by:** [@dani-garcia](https://github.com/dani-garcia)
**Base:** `main` ← **Head:** `fix-client-api-login-checks`
---
### 📝 Commits (1)
- [`1ca0d6e`](https://github.com/dani-garcia/vaultwarden/commit/1ca0d6e2451ae6f68b0325e7ae9f1233185bbb68) Validate all needed fields for client API login
### 📊 Changes
**1 file changed** (+4 additions, -0 deletions)
<details>
<summary>View changed files</summary>
📝 `src/api/identity.rs` (+4 -0)
</details>
### 📄 Description
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.
---
<sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
📋 Pull Request Information
Original PR: https://github.com/dani-garcia/vaultwarden/pull/3251
Author: @BlackDex
Created: 2/16/2023
Status: ✅ Merged
Merged: 2/21/2023
Merged by: @dani-garcia
Base:
main← Head:fix-client-api-login-checks📝 Commits (1)
1ca0d6eValidate all needed fields for client API login📊 Changes
1 file changed (+4 additions, -0 deletions)
View changed files
📝
src/api/identity.rs(+4 -0)📄 Description
During the client API login we need to have a
device_identifier,device_nameanddevice_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.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.