Remove signer override

It's no longer needed since AWS SDK v2 added a LegacyMd5Plugin
to ensure backward compatibility with S3-compliant storage.

Author: wendigo

lib/trino-filesystem-s3/src/main/java/io/trino/filesystem/s3/S3FileSystem.java

@@ -199,7 +199,6 @@ private void deleteObjects(Collection<Location> locations)
                         .overrideConfiguration(context::applyCredentialProviderOverride)
                         .requestPayer(requestPayer)
                         .bucket(bucket)
-                        .overrideConfiguration(disableStrongIntegrityChecksums())
                         .delete(builder -> builder.objects(objects).quiet(true))
                         .build();
 
@@ -394,14 +393,4 @@ private static void validateS3Location(Location location)
     {
         new S3Location(location);
     }
-
-    // TODO (https://github.com/trinodb/trino/issues/24955):
-    // remove me once all of the S3-compatible storage support strong integrity checks
-    @SuppressWarnings("deprecation")
-    static AwsRequestOverrideConfiguration disableStrongIntegrityChecksums()
-    {
-        return AwsRequestOverrideConfiguration.builder()
-            .signer(AwsS3V4Signer.create())
-            .build();
-    }
 }

lib/trino-filesystem-s3/src/test/java/io/trino/filesystem/s3/AbstractTestS3FileSystem.java

@@ -41,7 +41,6 @@
 
 import static com.google.common.collect.Iterables.getOnlyElement;
 import static io.trino.filesystem.encryption.EncryptionKey.randomAes256;
-import static io.trino.filesystem.s3.S3FileSystem.disableStrongIntegrityChecksums;
 import static io.trino.filesystem.s3.S3SseCUtils.encoded;
 import static io.trino.filesystem.s3.S3SseCUtils.md5Checksum;
 import static java.nio.charset.StandardCharsets.UTF_8;
@@ -157,7 +156,6 @@ void testFileWithTrailingWhitespaceAgainstNativeClient()
                             builder.sseCustomerKeyMD5(md5Checksum(randomEncryptionKey));
                         }
                     })
-                    .overrideConfiguration(disableStrongIntegrityChecksums())
                     .build();
 
             s3Client.putObject(putObjectRequest, RequestBody.fromBytes(contents.clone()));
@@ -273,9 +271,7 @@ public TempDirectory(S3Client s3Client, String path)
 
         public void create()
         {
-            s3Client.putObject(request -> request
-                    .overrideConfiguration(disableStrongIntegrityChecksums())
-                    .bucket(bucket()).key(path), RequestBody.empty());
+            s3Client.putObject(request -> request.bucket(bucket()).key(path), RequestBody.empty());
         }
 
         @Override

lib/trino-filesystem-s3/src/test/java/io/trino/filesystem/s3/TestS3FileSystemAwsS3.java

@@ -29,7 +29,6 @@
 import java.util.List;
 
 import static com.google.common.collect.Iterables.getOnlyElement;
-import static io.trino.filesystem.s3.S3FileSystem.disableStrongIntegrityChecksums;
 import static io.trino.testing.SystemEnvironmentUtils.requireEnv;
 import static org.assertj.core.api.Assertions.assertThat;
 
@@ -88,7 +87,6 @@ void testS3FileIteratorFileEntryTags()
                     .bucket(bucket())
                     .key(key)
                     .storageClass(storageClass.toString())
-                    .overrideConfiguration(disableStrongIntegrityChecksums())
                     .build();
             s3Client.putObject(
                     putObjectRequestBuilder,

testing/trino-testing-containers/src/main/java/io/trino/testing/containers/MotoContainer.java

@@ -17,12 +17,16 @@
 import software.amazon.awssdk.auth.credentials.AwsBasicCredentials;
 import software.amazon.awssdk.auth.credentials.StaticCredentialsProvider;
 import software.amazon.awssdk.awscore.client.builder.AwsClientBuilder;
+import software.amazon.awssdk.core.checksums.RequestChecksumCalculation;
 import software.amazon.awssdk.regions.Region;
+import software.amazon.awssdk.services.s3.LegacyMd5Plugin;
 import software.amazon.awssdk.services.s3.S3Client;
 import software.amazon.awssdk.services.s3.S3ClientBuilder;
 
 import java.net.URI;
 
+import static software.amazon.awssdk.core.checksums.ResponseChecksumValidation.WHEN_REQUIRED;
+
 public final class MotoContainer
         extends GenericContainer<MotoContainer>
 {
@@ -51,7 +55,10 @@ public void updateClient(AwsClientBuilder<?, ?> client)
                 AwsBasicCredentials.create(MOTO_ACCESS_KEY, MOTO_SECRET_KEY)));
         if (client instanceof S3ClientBuilder s3) {
             s3.forcePathStyle(true);
+            s3.responseChecksumValidation(WHEN_REQUIRED);
+            s3.requestChecksumCalculation(RequestChecksumCalculation.WHEN_REQUIRED);
         }
+        client.addPlugin(LegacyMd5Plugin.create());
     }
 
     public void createBucket(String bucketName)