-
Notifications
You must be signed in to change notification settings - Fork 83
Create a grace period before MFA is required #723
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small questions
ID: "00057-AddMFARequiredGracePeriod", | ||
Migrate: func(tx *gorm.DB) error { | ||
logger.Debugw("adding email verification required to realm") | ||
return tx.Exec("ALTER TABLE realms ADD COLUMN IF NOT EXISTS mfa_required_grace_period BIGINT DEFAULT 0").Error |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should the default be 0 or leave it as nullable?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sethvargo, whaught The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/hold |
New changes are detected. LGTM label has been removed. |
/unhold |
Issue #607
Proposed Changes
TODO: Update the registration page to show days remaining
Release Note