Skip to content

Move IsFIPSDisabledByEnv to lib/utils/aws/awsfips#52123

Merged
codingllama merged 1 commit intomasterfrom
codingllama/aws-fips-cycle
Feb 14, 2025
Merged

Move IsFIPSDisabledByEnv to lib/utils/aws/awsfips#52123
codingllama merged 1 commit intomasterfrom
codingllama/aws-fips-cycle

Conversation

@codingllama
Copy link
Copy Markdown
Contributor

Avoid an import cycle between lib/utils/aws/stsutils (and any other utils package) and lib/utils/aws.

The cycle doesn't happen in master, but it will happen in release branches. This PR helps to keep the code in sync.

@codingllama codingllama added the no-changelog Indicates that a PR does not require a changelog entry label Feb 13, 2025
@codingllama
Copy link
Copy Markdown
Contributor Author

No backport labels at it'll be cherry-picked as part of #52065 backports.

Comment thread lib/utils/aws/fips/fips_disabled.go Outdated
@codingllama codingllama force-pushed the codingllama/aws-fips-cycle branch from 227cb32 to 966555e Compare February 13, 2025 16:57
@codingllama codingllama changed the title Move IsFIPSDisabledByEnv to aws/fips Move IsFIPSDisabledByEnv to lib/utils/aws/awsfips Feb 13, 2025
@codingllama
Copy link
Copy Markdown
Contributor Author

Thanks for the quick reviews, folks!

@codingllama codingllama added this pull request to the merge queue Feb 13, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Feb 13, 2025
@codingllama codingllama added this pull request to the merge queue Feb 13, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Feb 13, 2025
@codingllama codingllama added this pull request to the merge queue Feb 13, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Feb 13, 2025
@codingllama codingllama added this pull request to the merge queue Feb 14, 2025
Merged via the queue into master with commit e374c24 Feb 14, 2025
@codingllama codingllama deleted the codingllama/aws-fips-cycle branch February 14, 2025 12:45
carloscastrojumo pushed a commit to carloscastrojumo/teleport that referenced this pull request Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog Indicates that a PR does not require a changelog entry size/sm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants