From aa07854f409c2246179259fef39e34a73ad0ce59 Mon Sep 17 00:00:00 2001 From: Zelda Hessler Date: Mon, 13 Feb 2023 17:32:59 -0600 Subject: [PATCH] Fix S3 canary match statement (#2358) * fix: s3 canary error match * update: use Error.is_ to check for "no such key" error in canary * remove: leftover import --- tools/ci-cdk/canary-lambda/src/s3_canary.rs | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tools/ci-cdk/canary-lambda/src/s3_canary.rs b/tools/ci-cdk/canary-lambda/src/s3_canary.rs index 830a76048..d08f9ab20 100644 --- a/tools/ci-cdk/canary-lambda/src/s3_canary.rs +++ b/tools/ci-cdk/canary-lambda/src/s3_canary.rs @@ -8,7 +8,6 @@ use crate::{mk_canary, CanaryEnv}; use anyhow::Context; use aws_config::SdkConfig; use aws_sdk_s3 as s3; -use s3::error::GetObjectError; use s3::types::ByteStream; use uuid::Uuid; @@ -35,12 +34,13 @@ pub async fn s3_canary(client: s3::Client, s3_bucket_name: String) -> anyhow::Re CanaryError(format!("Expected object {} to not exist in S3", test_key)).into(), ); } - Err(err) => match err.into_service_error() { - GetObjectError::NoSuchKey(..) => { - // good + Err(err) => { + let err = err.into_service_error(); + // If we get anything other than "No such key", we have a problem + if !err.is_no_such_key() { + return Err(err).context("unexpected s3::GetObject failure"); } - err => Err(err).context("unexpected s3::GetObject failure")?, - }, + } } // Put the test object -- GitLab