Avoid ClassCastException on every DynamoDB request in AwsSdkClientDecorator (#11285)

Avoid ClassCastException on every DynamoDB request in AwsSdkClientDecorator

`getValueForField("Key", String.class)` was called on every AWS SDK v2
request and a `ClassCastException` was caught on every DynamoDB
Get/Put/Update/Delete item call (Key is `Map<String, AttributeValue>`).
This forced the JVM to fill in a stack trace per request on a hot path.

Gate the `Key` extraction to S3 (the only service whose top-level `Key`
is a `String`), inline the helper, and drop the dead try/catch. Behavior
for S3 is unchanged — `setObjectKey` already wrote the same tag the
DSM-only branch was writing.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Merge branch 'master' into dougqh/aws-sdk-v2-skip-dynamodb-cce

Merge branch 'master' into dougqh/aws-sdk-v2-skip-dynamodb-cce

Merge branch 'master' into dougqh/aws-sdk-v2-skip-dynamodb-cce

Co-authored-by: devflow.devflow-routing-intake <devflow.devflow-routing-intake@kubernetes.us1.ddbuild.io>

Author: dougqh

dd-java-agent/instrumentation/aws-java/aws-java-sdk-2.2/src/main/java/datadog/trace/instrumentation/aws/v2/AwsSdkClientDecorator.java

@@ -133,14 +133,14 @@ public Context onSdkRequest(
 
     // S3
     request.getValueForField("Bucket", String.class).ifPresent(name -> setBucketName(span, name));
-    if ("s3".equalsIgnoreCase(awsServiceName) && traceConfig().isDataStreamsEnabled()) {
-      request
-          .getValueForField("Key", String.class)
-          .ifPresent(key -> span.setTag(InstrumentationTags.AWS_OBJECT_KEY, key));
-      span.setTag(Tags.HTTP_REQUEST_CONTENT_LENGTH, getRequestContentLength(httpRequest));
+    if ("s3".equalsIgnoreCase(awsServiceName)) {
+      // gate "Key" extraction to S3 — DynamoDB's Key is Map<String, AttributeValue>, would CCE
+      request.getValueForField("Key", String.class).ifPresent(key -> setObjectKey(span, key));
+      if (traceConfig().isDataStreamsEnabled()) {
+        span.setTag(Tags.HTTP_REQUEST_CONTENT_LENGTH, getRequestContentLength(httpRequest));
+      }
     }
 
-    getRequestKey(request).ifPresent(key -> setObjectKey(span, key));
     request
         .getValueForField("StorageClass", String.class)
         .ifPresent(
@@ -279,17 +279,6 @@ private static void setBucketName(AgentSpan span, String name) {
     setPeerService(span, InstrumentationTags.AWS_BUCKET_NAME, name);
   }
 
-  private static Optional<String> getRequestKey(SdkRequest request) {
-    Optional<String> key = Optional.empty();
-    try {
-      key = request.getValueForField("Key", String.class);
-    } catch (ClassCastException ignored) {
-      // Key is not always a string, like for dynamodb GetItemRequest
-    }
-
-    return key;
-  }
-
   private static void setObjectKey(AgentSpan span, String key) {
     span.setTag(InstrumentationTags.AWS_OBJECT_KEY, key);
   }