Skip to content

Commit

Permalink
Remove AWS_CREDENTIALS in backwards compatibility test (#5266)
Browse files Browse the repository at this point in the history
  • Loading branch information
gosar committed Jun 7, 2024
1 parent 5a9bd22 commit 9be3d6f
Showing 1 changed file with 6 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -75,20 +75,18 @@ public void beforeExecution(Context.BeforeExecution context, ExecutionAttributes
},
AwsSignerExecutionAttribute.SERVICE_SIGNING_NAME, // Endpoint rules override signing name
AwsSignerExecutionAttribute.SIGNING_REGION, // Endpoint rules override signing region
AwsSignerExecutionAttribute.AWS_CREDENTIALS, // Legacy auth strategy overrides credentials
AwsSignerExecutionAttribute.SIGNER_DOUBLE_URL_ENCODE); // Endpoint rules override double-url-encode
}

@Test
public void canSetSignerExecutionAttributes_modifyRequest() {
test(attributeModifications -> new ExecutionInterceptor() {
@Override
public SdkRequest modifyRequest(Context.ModifyRequest context, ExecutionAttributes executionAttributes) {
attributeModifications.accept(executionAttributes);
return context.request();
}
},
AwsSignerExecutionAttribute.AWS_CREDENTIALS); // Legacy auth strategy overrides credentials
@Override
public SdkRequest modifyRequest(Context.ModifyRequest context, ExecutionAttributes executionAttributes) {
attributeModifications.accept(executionAttributes);
return context.request();
}
});
}

@Test
Expand Down

0 comments on commit 9be3d6f

Please sign in to comment.