From d6b19bfe9914869e4eb8b2a99d6cb9fc0afb6686 Mon Sep 17 00:00:00 2001 From: Varun Dhananjaya Date: Fri, 13 Sep 2024 01:09:55 +0200 Subject: [PATCH] [identity] update transactwriteitems error log Summary: current logs do not tell us exactly what ddb error we're seeing. we need to know what the error is to understand why it's occurring Test Plan: locally confirmed that new log appears Reviewers: will, bartek Reviewed By: will Subscribers: ashoat, tomek Differential Revision: https://phab.comm.dev/D13314 --- services/identity/src/database/one_time_keys.rs | 1 + 1 file changed, 1 insertion(+) diff --git a/services/identity/src/database/one_time_keys.rs b/services/identity/src/database/one_time_keys.rs index a6864f008b..5ec10d60d8 100644 --- a/services/identity/src/database/one_time_keys.rs +++ b/services/identity/src/database/one_time_keys.rs @@ -141,6 +141,7 @@ impl DatabaseClient { match transaction { Ok(_) => return Ok((Some(otk_row.otk), requested_more_keys)), Err(e) => { + info!("Error retrieving one-time key: {:?}", e); let dynamo_db_error = DynamoDBError::from(e); let retryable_codes = HashSet::from([ retry::CONDITIONAL_CHECK_FAILED,