fix: prefer EndpointResolver Bucket for S3 in http label, AccountId for S3Control host label

[ganeshnj]

Issue #

Description of changes

S3 Bucket deduplication

Bucket in the HTTP URI is now a contextParam hence rules engine takes care of adding the bucket to the URI to either host name or path depending on the bucket location forceStylePath configuration. Therefore, SDK must not add the bucket to the URI outside of the rules engine.

"com.amazonaws.s3#GetObject": {
    "type": "operation",
    "input": {
        "target": "com.amazonaws.s3#GetObjectRequest"
    },
    "output": {
        "target": "com.amazonaws.s3#GetObjectOutput"
    },
    "traits": {
        "aws.protocols#httpChecksum": {
            "requestValidationModeMember": "ChecksumMode",
            "responseAlgorithms": [
                "CRC32",
                "CRC32C",
                "SHA256",
                "SHA1"
            ]
        },
        "smithy.api#http": {
            "method": "GET",
            "uri": "/{Bucket}/{Key+}?x-id=GetObject",
            "code": 200
        }
    }
},

"com.amazonaws.s3#GetObjectRequest": {
    "type": "structure",
    "members": {
        "Bucket": {
            "target": "com.amazonaws.s3#BucketName",
            "traits": {
                "smithy.api#httpLabel": {},
                "smithy.api#required": {},
                "smithy.rules#contextParam": {
                    "name": "Bucket"
                }
            }
        },

S3 Control AccountId deduplication

AccountId hostPrefix is now a staticContextParams hence rules engine takes care of adding AccountId to the host name. This is enabled by setting RequiresAccountId param in staticContextParams to true. Therefore, SDK must not add the AccountId to the host name outside of the rules engine.

"com.amazonaws.s3control#CreateAccessPoint": {
    "type": "operation",
    "input": {
        "target": "com.amazonaws.s3control#CreateAccessPointRequest"
    },
    "output": {
        "target": "com.amazonaws.s3control#CreateAccessPointResult"
    },
    "traits": {
        "smithy.api#endpoint": {
            "hostPrefix": "{AccountId}."
        },
        "smithy.api#http": {
            "method": "PUT",
            "uri": "/v20180820/accesspoint/{Name}",
            "code": 200
        },
        "smithy.rules#staticContextParams": {
            "RequiresAccountId": {
                "value": true
            }
        }
    }
}

New/existing dependencies impact assessment, if applicable

Conventional Commits

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[jbelkins]

Approved, but with a question