From b3a05277e8c1f4216916f34e13800d806e64d235 Mon Sep 17 00:00:00 2001
From: ryan <ryan@distrust.co>
Date: Sun, 4 Feb 2024 17:51:38 -0500
Subject: [PATCH] keyfork-shard: increase QR code read timeout from 30 to 60
 seconds

---
 crates/keyfork-shard/src/lib.rs     | 9 +++++----
 crates/keyfork-shard/src/openpgp.rs | 8 +++-----
 2 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/crates/keyfork-shard/src/lib.rs b/crates/keyfork-shard/src/lib.rs
index 614fd97..2b9e26c 100644
--- a/crates/keyfork-shard/src/lib.rs
+++ b/crates/keyfork-shard/src/lib.rs
@@ -44,6 +44,9 @@ pub struct InvalidData;
 pub(crate) const HUNK_VERSION: u8 = 1;
 pub(crate) const HUNK_OFFSET: usize = 2;
 
+const QRCODE_PROMPT: &str = "Press enter, then present QR code to camera.";
+const QRCODE_TIMEOUT: u64 = 60; // One minute
+
 /// Establish ECDH transport for remote operators, receive transport-encrypted shares, decrypt the
 /// shares, and combine them.
 ///
@@ -92,11 +95,9 @@ pub fn remote_decrypt(w: &mut impl Write) -> Result<(), Box<dyn std::error::Erro
 
         #[cfg(feature = "qrcode")]
         {
-            pm.prompt_message(PromptMessage::Text(
-                "Press enter, then present QR code to camera".to_string(),
-            ))?;
+            pm.prompt_message(PromptMessage::Text(QRCODE_PROMPT.to_string()))?;
             if let Ok(Some(hex)) =
-                keyfork_qrcode::scan_camera(std::time::Duration::from_secs(30), 0)
+                keyfork_qrcode::scan_camera(std::time::Duration::from_secs(QRCODE_TIMEOUT), 0)
             {
                 let decoded_data = smex::decode(&hex)?;
                 let _ = pubkey_data.insert(decoded_data[..32].try_into().map_err(|_| InvalidData)?);
diff --git a/crates/keyfork-shard/src/openpgp.rs b/crates/keyfork-shard/src/openpgp.rs
index 2ccf229..1abc11c 100644
--- a/crates/keyfork-shard/src/openpgp.rs
+++ b/crates/keyfork-shard/src/openpgp.rs
@@ -56,7 +56,7 @@ use smartcard::SmartcardManager;
 const SHARD_METADATA_VERSION: u8 = 1;
 const SHARD_METADATA_OFFSET: usize = 2;
 
-use super::{InvalidData, SharksError, HUNK_VERSION};
+use super::{InvalidData, SharksError, HUNK_VERSION, QRCODE_PROMPT, QRCODE_TIMEOUT};
 
 // 256 bit share is 49 bytes + some amount of hunk bytes, gives us reasonable padding
 const ENC_LEN: u8 = 4 * 16;
@@ -471,10 +471,8 @@ pub fn decrypt(
 
     #[cfg(feature = "qrcode")]
     {
-        pm.prompt_message(PromptMessage::Text(
-            "Press enter, then present QR code to camera".to_string(),
-        ))?;
-        if let Ok(Some(hex)) = keyfork_qrcode::scan_camera(std::time::Duration::from_secs(30), 0) {
+        pm.prompt_message(PromptMessage::Text(QRCODE_PROMPT.to_string()))?;
+        if let Ok(Some(hex)) = keyfork_qrcode::scan_camera(std::time::Duration::from_secs(QRCODE_TIMEOUT), 0) {
             let decoded_data = smex::decode(&hex)?;
             let _ = nonce_data.insert(decoded_data[..12].try_into().map_err(|_| InvalidData)?);
             let _ = pubkey_data.insert(decoded_data[12..].try_into().map_err(|_| InvalidData)?);