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

docs: add mfa_lifetime usecase #2253

Merged
merged 1 commit into from
Mar 21, 2025
Merged

docs: add mfa_lifetime usecase #2253

merged 1 commit into from
Mar 21, 2025

Conversation

pranav-okta
Copy link
Contributor

@pranav-okta pranav-okta commented Mar 20, 2025

The Terraform documentation for the okta_policy_rule_signon does not specify anything about the mfa_lifetime. The mfa_lifetime is only valid when the mfa_prompt is either ALWAYS or SESSION. Since this is not specified anywhere, there are issues where the mfa_lifetime is used even if the mfa_prompt is invalid. This PR adds docs about the specification.

@pranav-okta pranav-okta self-assigned this Mar 20, 2025
@pranav-okta pranav-okta requested a review from aditya-okta March 20, 2025 11:35
Copy link
Contributor

@aditya-okta aditya-okta left a comment

Choose a reason for hiding this comment

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

LGTM

@pranav-okta pranav-okta merged commit 65dceff into master Mar 21, 2025
3 checks passed
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