Skip to content

Commit

Permalink
[identity] update transactwriteitems error log
Browse files Browse the repository at this point in the history
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
  • Loading branch information
vdhanan committed Sep 13, 2024
1 parent ae6b513 commit d6b19bf
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions services/identity/src/database/one_time_keys.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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,
Expand Down

0 comments on commit d6b19bf

Please sign in to comment.