Skip to content

Commit b7f5371

Browse files
committed
Re-order the default credential providers
The EC2 provider uniquely has to make network requests even when not configured to determine failure, whereas all the other providers can quickly fail due to missing config. This generates delay and log noise when using one of the providers listed after EC2. Reordering them should continue to be fast for all providers including EC2, and improve the performance for eks and web_identity.
1 parent f4691cf commit b7f5371

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

README.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,9 @@ sources in the following default order:
1010
2. OS environment variables
1111
3. An AWS [credentials file][1]
1212
4. ECS task credentials
13-
5. EC2 metadata
14-
6. EKS Pod Identity
15-
7. Web Identity
13+
5. EKS Pod Identity
14+
6. Web Identity
15+
7. EC2 metadata
1616

1717
Usage
1818
-----

src/aws_credentials_provider.erl

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,9 +35,9 @@
3535
-type provider() :: aws_credentials_env
3636
| aws_credentials_file
3737
| aws_credentials_ecs
38-
| aws_credentials_ec2
3938
| aws_credentials_eks
4039
| aws_credentials_web_identity
40+
| aws_credentials_ec2
4141
| module().
4242
-type error_log() :: [{provider(), term()}].
4343
-export_type([ options/0, expiration/0, provider/0, error_log/0 ]).
@@ -50,9 +50,9 @@
5050
-define(DEFAULT_PROVIDERS, [aws_credentials_env,
5151
aws_credentials_file,
5252
aws_credentials_ecs,
53-
aws_credentials_ec2,
5453
aws_credentials_eks,
55-
aws_credentials_web_identity]).
54+
aws_credentials_web_identity,
55+
aws_credentials_ec2]).
5656

5757
-spec fetch() ->
5858
{ok, aws_credentials:credentials(), expiration()} |

0 commit comments

Comments
 (0)