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

feat: Allow non-FIPS endpoints on FIPS binaries #51924

Merged
merged 6 commits into from
Feb 11, 2025

Conversation

codingllama
Copy link
Contributor

@codingllama codingllama commented Feb 6, 2025

Allow FIPS binaries to use non-FIPS AWS endpoints of STS and DynamoDB. Useful for running in AWS regions that lack some of the FIPS services, but mostly ill-advised - talk to your FIPS auditors first.

The "escape hatch" is the TELEPORT_UNSTABLE_DISABLE_AWS_FIPS environment variable. Set it to yes|true|1 to enable it: TELEPORT_UNSTABLE_DISABLE_AWS_FIPS=yes.

"Forward"-port of #51932.

Changelog: Added an escape hatch to allow non-FIPS AWS endpoints on FIPS binaries (TELEPORT_UNSTABLE_DISABLE_AWS_FIPS=yes).

@codingllama codingllama changed the title Codingllama/aws sts fips feat: Add an escape hatch for non-FIPS STS on FIPS binaries Feb 6, 2025
@codingllama codingllama force-pushed the codingllama/aws-sts-fips branch from cf93ae6 to c44de74 Compare February 6, 2025 21:12
@codingllama codingllama changed the title feat: Add an escape hatch for non-FIPS STS on FIPS binaries feat: Allow non-FIPS endpoints on FIPS binaries Feb 10, 2025
@codingllama codingllama force-pushed the codingllama/aws-sts-fips branch 2 times, most recently from 127591b to 9858f97 Compare February 10, 2025 17:15
@codingllama codingllama marked this pull request as ready for review February 10, 2025 17:29
@github-actions github-actions bot added application-access audit-log Issues related to Teleports Audit Log database-access Database access related issues and PRs discovery kubernetes-access size/md labels Feb 10, 2025
@codingllama codingllama requested review from vapopov and removed request for creack February 10, 2025 17:31
@codingllama
Copy link
Contributor Author

Setting roughly the same reviewers as #51932.

@codingllama
Copy link
Contributor Author

@codingllama
Copy link
Contributor Author

Thanks, everyone! Fixing forbidigo lint woes, will queue soon. I'll also cherry-pick any changes to the v15 PR (#51932).

@codingllama codingllama added this pull request to the merge queue Feb 11, 2025
Merged via the queue into master with commit 7f0c4e9 Feb 11, 2025
43 checks passed
@codingllama codingllama deleted the codingllama/aws-sts-fips branch February 11, 2025 23:05
@public-teleport-github-review-bot

@codingllama See the table below for backport results.

Branch Result
branch/v16 Failed
branch/v17 Failed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants