Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: default credentials provider chain order #1324

Closed
wants to merge 5 commits into from

Conversation

0marperez
Copy link
Contributor

@0marperez 0marperez commented Jun 6, 2024

⚠️Replaced by #1434

Issue #

N/A

Description of changes

Change in the order of profile and STS web identity credentials providers in the default credentials provider chain

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

Copy link

github-actions bot commented Jun 6, 2024

A new generated diff is ready to view.

  • No codegen difference in the AWS SDK

This comment has been minimized.

@0marperez 0marperez marked this pull request as ready for review June 6, 2024 06:22
@0marperez 0marperez requested a review from a team as a code owner June 6, 2024 06:22
Copy link
Member

@lauzadis lauzadis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a behavior change, should we announce it to users in any way?

Copy link

A new generated diff is ready to view.

  • No codegen difference in the AWS SDK

Copy link

A new generated diff is ready to view.

  • No codegen difference in the AWS SDK

This comment has been minimized.

Copy link

Copy link

A new generated diff is ready to view.

  • No codegen difference in the AWS SDK

Copy link

Affected Artifacts

Changed in size
Artifact Pull Request (bytes) Latest Release (bytes) Delta (bytes) Delta (percentage)
ivschat-jvm.jar closure 8,913,945 8,913,939 6 0.00%
directconnect-jvm.jar closure 10,366,067 10,366,073 -6 -0.00%
paymentcryptography-jvm.jar closure 9,179,750 9,179,757 -7 -0.00%
directconnect-jvm.jar 2,137,396 2,137,404 -8 -0.00%
paymentcryptography-jvm.jar 951,079 951,088 -9 -0.00%
datazone-jvm.jar closure 13,246,684 13,246,830 -146 -0.00%
datazone-jvm.jar 5,018,013 5,018,161 -148 -0.00%
taxsettings-jvm.jar closure 8,975,963 8,976,686 -723 -0.01%
kinesisanalyticsv2-jvm.jar closure 10,668,896 10,669,952 -1,056 -0.01%
eks-jvm.jar closure 10,991,297 10,992,396 -1,099 -0.01%
globalaccelerator-jvm.jar closure 10,248,530 10,249,595 -1,065 -0.01%
s3-jvm.jar closure 12,815,843 12,817,239 -1,396 -0.01%
workspaces-jvm.jar closure 11,767,247 11,768,569 -1,322 -0.01%
organizations-jvm.jar closure 10,446,394 10,447,717 -1,323 -0.01%
emr-jvm.jar closure 11,162,888 11,165,712 -2,824 -0.03%
s3-jvm.jar 4,499,923 4,501,321 -1,398 -0.03%
acmpca-jvm.jar closure 9,431,962 9,435,191 -3,229 -0.03%
workspaces-jvm.jar 3,538,576 3,539,900 -1,324 -0.04%
batch-jvm.jar closure 10,285,570 10,289,441 -3,871 -0.04%
rekognition-jvm.jar closure 12,275,607 12,280,338 -4,731 -0.04%
eks-jvm.jar 2,762,626 2,763,727 -1,101 -0.04%
kinesisanalyticsv2-jvm.jar 2,440,225 2,441,283 -1,058 -0.04%
fsx-jvm.jar closure 11,447,888 11,453,432 -5,544 -0.05%
globalaccelerator-jvm.jar 2,019,859 2,020,926 -1,067 -0.05%
organizations-jvm.jar 2,217,723 2,219,048 -1,325 -0.06%
mediaconnect-jvm.jar closure 10,752,860 10,759,931 -7,071 -0.07%
emr-jvm.jar 2,934,217 2,937,043 -2,826 -0.10%
taxsettings-jvm.jar 747,292 748,017 -725 -0.10%
rekognition-jvm.jar 4,046,936 4,051,669 -4,733 -0.12%
fsx-jvm.jar 3,219,217 3,224,763 -5,546 -0.17%
qbusiness-jvm.jar closure 11,323,463 11,343,825 -20,362 -0.18%
batch-jvm.jar 2,056,899 2,060,772 -3,873 -0.19%
glue-jvm.jar closure 18,932,955 18,980,991 -48,036 -0.25%
acmpca-jvm.jar 1,203,291 1,206,522 -3,231 -0.27%
mediaconnect-jvm.jar 2,524,189 2,531,262 -7,073 -0.28%
ec2-jvm.jar closure 37,023,318 37,147,822 -124,504 -0.34%
paymentcryptographydata-jvm.jar closure 9,223,510 9,255,980 -32,470 -0.35%
medialive-jvm.jar closure 16,305,200 16,373,663 -68,463 -0.42%
ec2-jvm.jar 28,794,647 28,919,153 -124,506 -0.43%
glue-jvm.jar 10,704,284 10,752,322 -48,038 -0.45%
bedrock-jvm.jar closure 9,859,034 9,907,194 -48,160 -0.49%
arczonalshift-jvm.jar closure 8,806,734 8,854,621 -47,887 -0.54%
qbusiness-jvm.jar 3,007,543 3,027,907 -20,364 -0.67%
cloudhsmv2-jvm.jar closure 8,824,879 8,893,488 -68,609 -0.77%
medialive-jvm.jar 8,076,529 8,144,994 -68,465 -0.84%
firehose-jvm.jar closure 9,985,265 10,092,713 -107,448 -1.06%
bedrockruntime-jvm.jar closure 9,282,816 9,407,568 -124,752 -1.33%
sagemaker-jvm.jar closure 27,009,770 27,380,724 -370,954 -1.35%
connect-jvm.jar closure 18,896,617 19,209,617 -313,000 -1.63%
sagemaker-jvm.jar 18,781,099 19,152,055 -370,956 -1.94%
quicksight-jvm.jar closure 25,999,069 26,554,942 -555,873 -2.09%
licensemanagerlinuxsubscriptions-jvm.jar closure 8,516,993 8,720,542 -203,549 -2.33%
connect-jvm.jar 10,667,946 10,980,948 -313,002 -2.85%
bedrock-jvm.jar 1,630,363 1,678,525 -48,162 -2.87%
quicksight-jvm.jar 17,770,398 18,326,273 -555,875 -3.03%
paymentcryptographydata-jvm.jar 994,839 1,027,311 -32,472 -3.16%
bedrockagentruntime-jvm.jar closure 9,547,384 9,895,454 -348,070 -3.52%
firehose-jvm.jar 1,756,594 1,864,044 -107,450 -5.76%
arczonalshift-jvm.jar 578,063 625,952 -47,889 -7.65%
cloudhsmv2-jvm.jar 596,208 664,819 -68,611 -10.32%
bedrockruntime-jvm.jar 966,896 1,091,650 -124,754 -11.43%
bedrockagent-jvm.jar closure 10,180,346 11,763,354 -1,583,008 -13.46%
bedrockagentruntime-jvm.jar 1,231,464 1,579,536 -348,072 -22.04%
licensemanagerlinuxsubscriptions-jvm.jar 288,322 491,873 -203,551 -41.38%
bedrockagent-jvm.jar 1,951,675 3,534,685 -1,583,010 -44.79%
qapps-jvm.jar (does not exist) 1,024,547 -1,024,547 -100.00%
qapps-jvm.jar closure (does not exist) 9,253,216 -9,253,216 -100.00%

@0marperez 0marperez closed this Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants