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

Update AWS config to fix SSO login #53

Merged
merged 1 commit into from
Mar 3, 2024
Merged

Conversation

petrzjunior
Copy link
Contributor

what

  • update aws/aws-sdk-go-v2/config and aws/aws-sdk-go-v2/credentials to the newest versions

why

Error: error configuring Terraform AWS Provider: loading configuration: profile "profile" is configured to use SSO but is missing required configuration: sso_region, sso_start_url

references

P.S. This is my first ever Go PR, feel free to correct me. This is currently a blocking issue for me so I am trying to push it.

@petrzjunior petrzjunior requested review from a team as code owners January 1, 2024 21:14
@rafaljanicki
Copy link

Any chance to get this merged @mcalhoun ?

@sebrichards
Copy link

Perhaps @Nuru you'd be able to review this?

@petrzjunior
Copy link
Contributor Author

Btw. I have compiled it locally and been using it successfully for 2 months without any problems.

Copy link
Member

@aknysh aknysh left a comment

Choose a reason for hiding this comment

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

LGTM

@hans-d hans-d merged commit 87e804a into cloudposse:main Mar 3, 2024
@miljanic
Copy link

miljanic commented Mar 4, 2024

@aknysh @hans-d Do you guys have a plan for when this version will be available on the terraform registry?

@hans-d
Copy link

hans-d commented Mar 4, 2024

We're currently checking - as some repos dont seem to be having a working auto-update-me for the registry.

@rafaljanicki
Copy link

Any updates here @hans-d ? I see that the TF registry still has the old version

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.

SSO authentication is not supported
6 participants