vw_small

commit 82fee0ede36dacc089b9f6ff0c06816338cdb7c5
parent 9254cf9d9c1b43a4ad3bc640610048c0a798424e
Author: Mathijs van Veluw <black.dex@gmail.com>
Date:   Sun, 20 Jun 2021 18:07:18 +0200

Merge pull request #1779 from jjlin/last-known-rev-warning

Avoid `Error parsing LastKnownRevisionDate` warning for mobile clients
Diffstat:
M
src/api/core/ciphers.rs
 | 
8
+++++++-

1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/src/api/core/ciphers.rs b/src/api/core/ciphers.rs
@@ -268,7 +268,13 @@ fn post_ciphers_create(data: JsonUpcase<ShareCipherData>, headers: Headers, conn
 /// Called when creating a new user-owned cipher.
 #[post("/ciphers", data = "<data>")]
 fn post_ciphers(data: JsonUpcase<CipherData>, headers: Headers, conn: DbConn, nt: Notify) -> JsonResult {
-    let data: CipherData = data.into_inner().data;
+    let mut data: CipherData = data.into_inner().data;
+
+    // The web/browser clients set this field to null as expected, but the
+    // mobile clients seem to set the invalid value `0001-01-01T00:00:00`,
+    // which results in a warning message being logged. This field isn't
+    // needed when creating a new cipher, so just ignore it unconditionally.
+    data.LastKnownRevisionDate = None;
 
     let mut cipher = Cipher::new(data.Type, data.Name.clone());
     update_cipher_from_data(&mut cipher, data, &headers, false, &conn, &nt, UpdateType::CipherCreate)?;