Skip to content

RFD 01111 - Support connection testers when per-session MFA is enabled#22458

Merged
kimlisa merged 2 commits intomasterfrom
rfd/0111-user-cert-accepts-mfa-authn-response
Mar 3, 2023
Merged

RFD 01111 - Support connection testers when per-session MFA is enabled#22458
kimlisa merged 2 commits intomasterfrom
rfd/0111-user-cert-accepts-mfa-authn-response

Conversation

@kimlisa
Copy link
Copy Markdown
Contributor

@kimlisa kimlisa commented Mar 1, 2023

@github-actions github-actions Bot requested review from jakule and tobiaszheller March 1, 2023 07:22
@github-actions github-actions Bot added rfd Request for Discussion size/sm labels Mar 1, 2023
@kimlisa kimlisa requested review from codingllama and marcoandredinis and removed request for jakule and tobiaszheller March 1, 2023 07:24
state: draft
---

# RFD 0111 - GenerateUserCerts accepts and validates a MFA authentication response
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This title seems more like an implementation detail than a description of the proposed change.

How about something like:

Suggested change
# RFD 0111 - GenerateUserCerts accepts and validates a MFA authentication response
# RFD 0111 - Support connection testers when per-session MFA is enabled

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Same goes for the PR title :-)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ah right thanks!

@kimlisa kimlisa changed the title RFD 01111 - GenerateUserCerts accepts and validates a MFA authentication response RFD 01111 - Support connection testers when per-session MFA is enabled Mar 1, 2023
Copy link
Copy Markdown
Contributor

@codingllama codingllama left a comment

Choose a reason for hiding this comment

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

Thanks for the RFD, Lisa. Short and sweet.

@kimlisa kimlisa added this pull request to the merge queue Mar 3, 2023
Merged via the queue into master with commit 5aa3a41 Mar 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rfd Request for Discussion size/sm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants