Generate ListBucketMetricsConfiguration

[peterrsongg]

Description

Continuation of #4102

Motivation and Context

Generates ListBucketMetricsConfiguration

Testing

All testng done in larger PR with all 4 operations which this PR is based off of.
dry run passed

Screenshots (if appropriate)

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist

• My code follows the code style of this project
• My change requires a change to the documentation
• I have updated the documentation accordingly
• I have read the README document
• I have added tests to cover my changes
• All new and existing tests passed

License

• I confirm that this pull request can be released under the Apache 2 license

[peterrsongg]

I'm pasting the results of the AI analysis done on the larger PR here
Updated AI prompt run on latest revision.

The added validation isn't a breaking change because the request will fail and throw AmazonS3Exception and since we are throwing the same exception but just earlier in the api call it isn't a breaking change. I intentionally made it the base service exception instead of a System.ArgumentException for that reason.

Now I have all the information needed to provide a comprehensive analysis. Let me compare the deleted custom files with the generated ones and examine all the marshaller changes:

Task Completed

Breaking Changes Analysis - S3 Custom to Generated Migration

After analyzing all deleted custom files, their generated replacements, and modified marshallers, I have identified the following breaking changes:

BREAKING CHANGES IDENTIFIED:

1. AbortMultipartUploadRequestMarshaller.cs - NEW VALIDATION ADDED

Issue: Added new required field validation that didn't exist before:

if (string.IsNullOrEmpty(publicRequest.UploadId))
    throw new AmazonS3Exception("Request object does not have required field UploadId set");

Impact: Customer code that previously could make AbortMultipartUpload requests without UploadId set will now throw an exception.

2. CopyPartRequestMarshaller.cs - NEW VALIDATIONS ADDED

Issue: Added two new required field validations:

if (publicRequest.PartNumber == null)
    throw new AmazonS3Exception("Request object does not have required field PartNumber set");
if (string.IsNullOrEmpty(publicRequest.UploadId))
    throw new AmazonS3Exception("Request object does not have required field UploadId set");

Impact: Customer code that previously could make CopyPart requests without PartNumber or UploadId set will now throw exceptions.

3. ListPartsRequestMarshaller.cs - NEW VALIDATION ADDED

Issue: Added new required field validation:

if (string.IsNullOrEmpty(publicRequest.UploadId))
    throw new AmazonS3Exception("Request object does not have required field UploadId set");

Impact: Customer code that previously could make ListParts requests without UploadId set will now throw an exception.

NO BREAKING CHANGES FOUND IN:

Model Files Migration (✅ SAFE)

• DeleteBucketMetricsConfiguration Request/Response
• GetBucketMetricsConfiguration Request/Response
• ListBucketMetricsConfigurations Request/Response
• MetricsConfiguration
• PutBucketMetricsConfiguration Request/Response

Differences are non-breaking:

• Private field naming changes (bucketName → _bucketName)
• Added [AWSProperty(Required=true)] attributes (not breaking for existing code)
• Minor documentation text changes
• Collection initialization patterns updated to newer SDK standards

Marshaller Logic Preservation (✅ SAFE)

• GetBucketMetricsConfigurationRequestMarshaller: Custom PreMarshallCustomization (Suppress404Exceptions) properly preserved
• PutBucketMetricsConfigurationRequestMarshaller: Custom MetricsFilterCustomMarshall and PostMarshallCustomization (checksum logic) properly preserved through partial methods
• PutLifecycleConfigurationRequestMarshaller: Filter marshalling logic properly moved to FilterCustomMarshall method

SUMMARY:

• Files analyzed: 30 out of 30 total changed files
• Breaking changes found: 3 files with new required field validations
• Safe migrations: 27 files with logic properly preserved

The breaking changes are all related to new validation logic being added to existing marshallers rather than logic being lost during migration. These validations enforce that required fields are set before making API calls, which technically makes the SDK more strict than before.

[dscpinheiro]

Was this supposed to be in this PR?

[peterrsongg]

no, that should only be in the very first PR. this change is in both branches so not sure why it is showing up here😅

[peterrsongg]

okay i fixed it

[dscpinheiro]

Similar to the other PR, we were doing sub resources before:

aws-sdk-net/sdk/src/Services/S3/Custom/Model/Internal/MarshallTransformations/ListBucketMetricsConfigurationsRequestMarshaller.cs

Line 51 in 62c261a

request.AddSubResource("continuation-token", listBucketMetricsConfigurationRequest.ContinuationToken.ToString());

Is that intentional here too?

[peterrsongg]

yup, that is intentional. We were incorrectly adding these non static-query parameters to the subresources collection, even though they are modeled as querystring. 99% of the time it is fine, but when there are special characters in any of these query strings (for example a datetime), we wont percent encode it in the URL which is incorrect. Moving it to the Parameters section where it belongs ensures it is percent encoded in the url