From 28f3594de3fb6b8bb2ce824438a3d69dbda1ace5 Mon Sep 17 00:00:00 2001 From: bunnie Date: Thu, 26 Jan 2023 17:59:52 +0800 Subject: [PATCH] make reset timing a bit looser currently 10 seconds from boot before cred reset can be issued; automated testing can't always keep up with that. expand to 20 seconds. This is not "standards compliant" but the idea that you unplug the device and it stays on is also not "standards compliant". --- apps/vault/src/ctap/mod.rs | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/apps/vault/src/ctap/mod.rs b/apps/vault/src/ctap/mod.rs index 2ed1db10c..bd8bfb0b9 100644 --- a/apps/vault/src/ctap/mod.rs +++ b/apps/vault/src/ctap/mod.rs @@ -115,7 +115,8 @@ pub const TOUCH_TIMEOUT: Duration = Duration::from_millis(TOUCH_TIMEOUT_MS); #[cfg(feature = "with_ctap1")] pub const U2F_UP_PROMPT_TIMEOUT: Duration = Duration::from_millis(10000); // TODO(kaczmarczyck) 2.1 allows Reset after Reset and 15 seconds? -const RESET_TIMEOUT_DURATION: Duration = Duration::from_millis(10000); +// Needs more tolerance for Xous automated test bench, expanded to 20 seconds. +const RESET_TIMEOUT_DURATION: Duration = Duration::from_millis(20000); const STATEFUL_COMMAND_TIMEOUT_DURATION: Duration = Duration::from_millis(30000); pub const FIDO2_VERSION_STRING: &str = "FIDO_2_0";