aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBlackDex <[email protected]>2021-12-31 15:59:58 +0100
committerBlackDex <[email protected]>2021-12-31 15:59:58 +0100
commitbf5aefd12976cef55276e00db770710b22d7b62f (patch)
tree8abf8d2bc09fe4da1a58e62c2848c1328be05a64
parent920371929bc89f5000d26c55b834148b32c8058f (diff)
downloadvaultwarden-bf5aefd12976cef55276e00db770710b22d7b62f.tar.gz
vaultwarden-bf5aefd12976cef55276e00db770710b22d7b62f.zip
Fix issue with Bitwarden CLI.
The CLI seems to send a String instead of an Integer for the maximum access count. It now accepts both types and converts it to an i32 in all cases. Fixes #2196
-rw-r--r--src/api/core/sends.rs38
1 files changed, 22 insertions, 16 deletions
diff --git a/src/api/core/sends.rs b/src/api/core/sends.rs
index aa4fd792..72437f15 100644
--- a/src/api/core/sends.rs
+++ b/src/api/core/sends.rs
@@ -7,7 +7,7 @@ use rocket_contrib::json::Json;
use serde_json::Value;
use crate::{
- api::{ApiResult, EmptyResult, JsonResult, JsonUpcase, Notify, UpdateType},
+ api::{ApiResult, EmptyResult, JsonResult, JsonUpcase, Notify, NumberOrString, UpdateType},
auth::{Headers, Host},
db::{models::*, DbConn, DbPool},
util::SafeString,
@@ -42,21 +42,21 @@ pub fn purge_sends(pool: DbPool) {
#[derive(Deserialize)]
#[allow(non_snake_case)]
-pub struct SendData {
- pub Type: i32,
- pub Key: String,
- pub Password: Option<String>,
- pub MaxAccessCount: Option<i32>,
- pub ExpirationDate: Option<DateTime<Utc>>,
- pub DeletionDate: DateTime<Utc>,
- pub Disabled: bool,
- pub HideEmail: Option<bool>,
+struct SendData {
+ Type: i32,
+ Key: String,
+ Password: Option<String>,
+ MaxAccessCount: Option<NumberOrString>,
+ ExpirationDate: Option<DateTime<Utc>>,
+ DeletionDate: DateTime<Utc>,
+ Disabled: bool,
+ HideEmail: Option<bool>,
// Data field
- pub Name: String,
- pub Notes: Option<String>,
- pub Text: Option<Value>,
- pub File: Option<Value>,
+ Name: String,
+ Notes: Option<String>,
+ Text: Option<Value>,
+ File: Option<Value>,
}
/// Enforces the `Disable Send` policy. A non-owner/admin user belonging to
@@ -119,7 +119,10 @@ fn create_send(data: SendData, user_uuid: String) -> ApiResult<Send> {
let mut send = Send::new(data.Type, data.Name, data_str, data.Key, data.DeletionDate.naive_utc());
send.user_uuid = Some(user_uuid);
send.notes = data.Notes;
- send.max_access_count = data.MaxAccessCount;
+ send.max_access_count = match data.MaxAccessCount {
+ Some(m) => Some(m.into_i32()?),
+ _ => None,
+ };
send.expiration_date = data.ExpirationDate.map(|d| d.naive_utc());
send.disabled = data.Disabled;
send.hide_email = data.HideEmail;
@@ -414,7 +417,10 @@ fn put_send(id: String, data: JsonUpcase<SendData>, headers: Headers, conn: DbCo
send.akey = data.Key;
send.deletion_date = data.DeletionDate.naive_utc();
send.notes = data.Notes;
- send.max_access_count = data.MaxAccessCount;
+ send.max_access_count = match data.MaxAccessCount {
+ Some(m) => Some(m.into_i32()?),
+ _ => None,
+ };
send.expiration_date = data.ExpirationDate.map(|d| d.naive_utc());
send.hide_email = data.HideEmail;
send.disabled = data.Disabled;