smithy-lang / smithy-swift

Apache License 2.0
28 stars 27 forks source link

feat: finish interceptors migration #781

Closed milesziemer closed 2 months ago

milesziemer commented 2 months ago

Description of changes

aws-sdk-swift pr: https://github.com/awslabs/aws-sdk-swift/pull/1630

Removes all Middleware types, references, and implementations, completing migration to using Orchestrator + Interceptors. Tests using middleware were also migrated to use Orchestrator + Interceptors. Codegen was updated, removing the useInterceptors setting, switching all code that was either generating middleware or interceptors to just generate interceptors. This also allowed removing some other existing generator code around middleware steps.

As with the corresponding aws-sdk-swift commit, this is already a large commit so I wanted to avoid doing too much refactoring or moving code around. For example, I didn't change file names or type names. There's probably a good deal of refactoring that can be done, but it is out of scope for this PR.

Also, this commit removes the amz-sdk-invocation-id/amz-sdk-request header impl as it was moved to aws-sdk-swift in the corresponding commit.

Scope

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

milesziemer commented 2 months ago

Closing for https://github.com/smithy-lang/smithy-swift/pull/782 so CI will run