fix: move endpoint resolution to the serializers

[AllanZhengYP]

Fix a regression introduced in #1097
Paired with smithy-lang/smithy-typescript#162

Previously, client.config.endpoint, which is a Provider<Endpoint> is resolved in serializer middleware:

aws-sdk-js-v3/packages/middleware-serde/src/serializerMiddleware.ts

Lines 23 to 26 in fde80bf

	const endpointResolvedOptions = {
	...options,
	endpoint: await options.endpoint()
	};

. But the type is not changed. As a result, the SerDeContext.endpoint and actually an Endpoint but typed in Provider<Endpoint>.

This change move the endpoint resolution to the serializer itself SerDeContext.endpoint is a provider function accross the board.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[codecov-io]

Codecov Report

Merging #1106 into master will decrease coverage by 5.25%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1106      +/-   ##
==========================================
- Coverage   78.79%   73.54%   -5.26%     
==========================================
  Files         279      283       +4     
  Lines       11599    13014    +1415     
  Branches     2402     3030     +628     
==========================================
+ Hits         9140     9571     +431     
- Misses       2459     3443     +984     

Impacted Files	Coverage Δ
packages/middleware-stack/src/MiddlewareStack.ts	88.61% <ø> (ø)
protocol_tests/aws-ec2/EC2ProtocolClient.ts	96.00% <ø> (ø)
...ws-ec2/commands/EmptyInputAndEmptyOutputCommand.ts	100.00% <ø> (ø)
...ests/aws-ec2/commands/GreetingWithErrorsCommand.ts	100.00% <ø> (ø)
.../aws-ec2/commands/IgnoresWrappingXmlNameCommand.ts	100.00% <ø> (ø)
..._tests/aws-ec2/commands/NestedStructuresCommand.ts	88.23% <ø> (ø)
..._tests/aws-ec2/commands/NoInputAndOutputCommand.ts	100.00% <ø> (ø)
...2/commands/QueryIdempotencyTokenAutoFillCommand.ts	88.23% <ø> (ø)
...otocol_tests/aws-ec2/commands/QueryListsCommand.ts	88.23% <ø> (ø)
...l_tests/aws-ec2/commands/QueryTimestampsCommand.ts	88.23% <ø> (ø)
... and 170 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fde80bf...1e7922c. Read the comment docs.

[aws-sdk-js-automation]

AWS CodeBuild CI Report

• CodeBuild project: sdk-staging-test
• Commit ID: 1e7922c
• Result: SUCCEEDED
• Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.