add missing log

smallrye · Feb 26, 2024 · a1b7f6f · a1b7f6f
1 parent bd4fa6a
commit a1b7f6f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/...ive-messaging-aws-sqs/src/main/java/io/smallrye/reactive/messaging/aws/sqs/SqsConfig.java b/...ive-messaging-aws-sqs/src/main/java/io/smallrye/reactive/messaging/aws/sqs/SqsConfig.java
@@ -40,7 +40,7 @@ public Optional<Region> getRegion() {
         try {
             return Optional.of(Region.of(result.get()));
         } catch (IllegalArgumentException e) {
-            // TODO: LOG
+            log.failedToParseAwsRegion(result.get(), e.getMessage());
             return Optional.empty();
         }
     }

diff --git a/...ging-aws-sqs/src/main/java/io/smallrye/reactive/messaging/aws/sqs/i18n/AwsSqsLogging.java b/...ging-aws-sqs/src/main/java/io/smallrye/reactive/messaging/aws/sqs/i18n/AwsSqsLogging.java
@@ -30,4 +30,8 @@ public interface AwsSqsLogging extends BasicLogger {
     @LogMessage(level = Level.WARN)
     @Message(id = 19303, value = "Failed to load the AWS credential loader, use the default credential provider chain %s")
     void failedToLoadAwsCredentialLoader(String message);
+
+    @LogMessage(level = Level.WARN)
+    @Message(id = 19304, value = "Failed to parse provided AWS Region, will apply the AWS SDK built-in logic to detect region. %s, %s")
+    void failedToParseAwsRegion(String region, String message);
 }