feat(middleware-stack): allow adding middleware to override an existing one (#1964)

* feat(middleware-stack): allow adding middleware to override an existing one

* fix: add override config to all native middleware

* fix: remove resolved flag in commands(revert #1883)

This reverts commit d4c302b.

Author: AllanZhengYP

clients/client-accessanalyzer/commands/CreateAnalyzerCommand.ts

@@ -28,7 +28,6 @@ export class CreateAnalyzerCommand extends $Command<
   CreateAnalyzerCommandOutput,
   AccessAnalyzerClientResolvedConfig
 > {
-  private resolved = false;
   // Start section: command_properties
   // End section: command_properties
 
@@ -46,10 +45,7 @@ export class CreateAnalyzerCommand extends $Command<
     configuration: AccessAnalyzerClientResolvedConfig,
     options?: __HttpHandlerOptions
   ): Handler<CreateAnalyzerCommandInput, CreateAnalyzerCommandOutput> {
-    if (!this.resolved) {
-      this.middlewareStack.use(getSerdePlugin(configuration, this.serialize, this.deserialize));
-      this.resolved = true;
-    }
+    this.middlewareStack.use(getSerdePlugin(configuration, this.serialize, this.deserialize));
 
     const stack = clientStack.concat(this.middlewareStack);
 

clients/client-accessanalyzer/commands/CreateArchiveRuleCommand.ts

@@ -29,7 +29,6 @@ export class CreateArchiveRuleCommand extends $Command<
   CreateArchiveRuleCommandOutput,
   AccessAnalyzerClientResolvedConfig
 > {
-  private resolved = false;
   // Start section: command_properties
   // End section: command_properties
 
@@ -47,10 +46,7 @@ export class CreateArchiveRuleCommand extends $Command<
     configuration: AccessAnalyzerClientResolvedConfig,
     options?: __HttpHandlerOptions
   ): Handler<CreateArchiveRuleCommandInput, CreateArchiveRuleCommandOutput> {
-    if (!this.resolved) {
-      this.middlewareStack.use(getSerdePlugin(configuration, this.serialize, this.deserialize));
-      this.resolved = true;
-    }
+    this.middlewareStack.use(getSerdePlugin(configuration, this.serialize, this.deserialize));
 
     const stack = clientStack.concat(this.middlewareStack);
 

clients/client-accessanalyzer/commands/DeleteAnalyzerCommand.ts

@@ -30,7 +30,6 @@ export class DeleteAnalyzerCommand extends $Command<
   DeleteAnalyzerCommandOutput,
   AccessAnalyzerClientResolvedConfig
 > {
-  private resolved = false;
   // Start section: command_properties
   // End section: command_properties
 
@@ -48,10 +47,7 @@ export class DeleteAnalyzerCommand extends $Command<
     configuration: AccessAnalyzerClientResolvedConfig,
     options?: __HttpHandlerOptions
   ): Handler<DeleteAnalyzerCommandInput, DeleteAnalyzerCommandOutput> {
-    if (!this.resolved) {
-      this.middlewareStack.use(getSerdePlugin(configuration, this.serialize, this.deserialize));
-      this.resolved = true;
-    }
+    this.middlewareStack.use(getSerdePlugin(configuration, this.serialize, this.deserialize));
 
     const stack = clientStack.concat(this.middlewareStack);
 

clients/client-accessanalyzer/commands/DeleteArchiveRuleCommand.ts

@@ -28,7 +28,6 @@ export class DeleteArchiveRuleCommand extends $Command<
   DeleteArchiveRuleCommandOutput,
   AccessAnalyzerClientResolvedConfig
 > {
-  private resolved = false;
   // Start section: command_properties
   // End section: command_properties
 
@@ -46,10 +45,7 @@ export class DeleteArchiveRuleCommand extends $Command<
     configuration: AccessAnalyzerClientResolvedConfig,
     options?: __HttpHandlerOptions
   ): Handler<DeleteArchiveRuleCommandInput, DeleteArchiveRuleCommandOutput> {
-    if (!this.resolved) {
-      this.middlewareStack.use(getSerdePlugin(configuration, this.serialize, this.deserialize));
-      this.resolved = true;
-    }
+    this.middlewareStack.use(getSerdePlugin(configuration, this.serialize, this.deserialize));
 
     const stack = clientStack.concat(this.middlewareStack);
 

clients/client-accessanalyzer/commands/GetAnalyzedResourceCommand.ts

@@ -28,7 +28,6 @@ export class GetAnalyzedResourceCommand extends $Command<
   GetAnalyzedResourceCommandOutput,
   AccessAnalyzerClientResolvedConfig
 > {
-  private resolved = false;
   // Start section: command_properties
   // End section: command_properties
 
@@ -46,10 +45,7 @@ export class GetAnalyzedResourceCommand extends $Command<
     configuration: AccessAnalyzerClientResolvedConfig,
     options?: __HttpHandlerOptions
   ): Handler<GetAnalyzedResourceCommandInput, GetAnalyzedResourceCommandOutput> {
-    if (!this.resolved) {
-      this.middlewareStack.use(getSerdePlugin(configuration, this.serialize, this.deserialize));
-      this.resolved = true;
-    }
+    this.middlewareStack.use(getSerdePlugin(configuration, this.serialize, this.deserialize));
 
     const stack = clientStack.concat(this.middlewareStack);
 

clients/client-accessanalyzer/commands/GetAnalyzerCommand.ts

@@ -28,7 +28,6 @@ export class GetAnalyzerCommand extends $Command<
   GetAnalyzerCommandOutput,
   AccessAnalyzerClientResolvedConfig
 > {
-  private resolved = false;
   // Start section: command_properties
   // End section: command_properties
 
@@ -46,10 +45,7 @@ export class GetAnalyzerCommand extends $Command<
     configuration: AccessAnalyzerClientResolvedConfig,
     options?: __HttpHandlerOptions
   ): Handler<GetAnalyzerCommandInput, GetAnalyzerCommandOutput> {
-    if (!this.resolved) {
-      this.middlewareStack.use(getSerdePlugin(configuration, this.serialize, this.deserialize));
-      this.resolved = true;
-    }
+    this.middlewareStack.use(getSerdePlugin(configuration, this.serialize, this.deserialize));
 
     const stack = clientStack.concat(this.middlewareStack);
 

clients/client-accessanalyzer/commands/GetArchiveRuleCommand.ts

@@ -28,7 +28,6 @@ export class GetArchiveRuleCommand extends $Command<
   GetArchiveRuleCommandOutput,
   AccessAnalyzerClientResolvedConfig
 > {
-  private resolved = false;
   // Start section: command_properties
   // End section: command_properties
 
@@ -46,10 +45,7 @@ export class GetArchiveRuleCommand extends $Command<
     configuration: AccessAnalyzerClientResolvedConfig,
     options?: __HttpHandlerOptions
   ): Handler<GetArchiveRuleCommandInput, GetArchiveRuleCommandOutput> {
-    if (!this.resolved) {
-      this.middlewareStack.use(getSerdePlugin(configuration, this.serialize, this.deserialize));
-      this.resolved = true;
-    }
+    this.middlewareStack.use(getSerdePlugin(configuration, this.serialize, this.deserialize));
 
     const stack = clientStack.concat(this.middlewareStack);
 

clients/client-accessanalyzer/commands/GetFindingCommand.ts

@@ -28,7 +28,6 @@ export class GetFindingCommand extends $Command<
   GetFindingCommandOutput,
   AccessAnalyzerClientResolvedConfig
 > {
-  private resolved = false;
   // Start section: command_properties
   // End section: command_properties
 
@@ -46,10 +45,7 @@ export class GetFindingCommand extends $Command<
     configuration: AccessAnalyzerClientResolvedConfig,
     options?: __HttpHandlerOptions
   ): Handler<GetFindingCommandInput, GetFindingCommandOutput> {
-    if (!this.resolved) {
-      this.middlewareStack.use(getSerdePlugin(configuration, this.serialize, this.deserialize));
-      this.resolved = true;
-    }
+    this.middlewareStack.use(getSerdePlugin(configuration, this.serialize, this.deserialize));
 
     const stack = clientStack.concat(this.middlewareStack);
 

clients/client-accessanalyzer/commands/ListAnalyzedResourcesCommand.ts

@@ -29,7 +29,6 @@ export class ListAnalyzedResourcesCommand extends $Command<
   ListAnalyzedResourcesCommandOutput,
   AccessAnalyzerClientResolvedConfig
 > {
-  private resolved = false;
   // Start section: command_properties
   // End section: command_properties
 
@@ -47,10 +46,7 @@ export class ListAnalyzedResourcesCommand extends $Command<
     configuration: AccessAnalyzerClientResolvedConfig,
     options?: __HttpHandlerOptions
   ): Handler<ListAnalyzedResourcesCommandInput, ListAnalyzedResourcesCommandOutput> {
-    if (!this.resolved) {
-      this.middlewareStack.use(getSerdePlugin(configuration, this.serialize, this.deserialize));
-      this.resolved = true;
-    }
+    this.middlewareStack.use(getSerdePlugin(configuration, this.serialize, this.deserialize));
 
     const stack = clientStack.concat(this.middlewareStack);
 

clients/client-accessanalyzer/commands/ListAnalyzersCommand.ts

@@ -28,7 +28,6 @@ export class ListAnalyzersCommand extends $Command<
   ListAnalyzersCommandOutput,
   AccessAnalyzerClientResolvedConfig
 > {
-  private resolved = false;
   // Start section: command_properties
   // End section: command_properties
 
@@ -46,10 +45,7 @@ export class ListAnalyzersCommand extends $Command<
     configuration: AccessAnalyzerClientResolvedConfig,
     options?: __HttpHandlerOptions
   ): Handler<ListAnalyzersCommandInput, ListAnalyzersCommandOutput> {
-    if (!this.resolved) {
-      this.middlewareStack.use(getSerdePlugin(configuration, this.serialize, this.deserialize));
-      this.resolved = true;
-    }
+    this.middlewareStack.use(getSerdePlugin(configuration, this.serialize, this.deserialize));
 
     const stack = clientStack.concat(this.middlewareStack);