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

Update docs/guides to reference MFA #24689

Merged
merged 6 commits into from
Dec 12, 2024
Merged

Conversation

iansltx
Copy link
Member

@iansltx iansltx commented Dec 12, 2024

For #23910; docs for #22078.

Also fixes "does Fleet supported" typo.

For #23910; docs for #22078.

Also fixes "does Fleet supported" typo
@iansltx
Copy link
Member Author

iansltx commented Dec 12, 2024

@rachaelshaw I thought about mentioning emails as a soft dependency on the reference architecture page, using somewhat similar language to what I added in the email backends area of server config. LMK if you want me to try my hand at that, or if you want to back out the server config mods feel free to revert that bit.

I didn't see any articles specifically talking about auth that weren't in the context of SSO, so the changes here wind up being minimal.

Swapped the MFA link in the pricing table as 22078 feels a bit more well-groomed for explaining what got implemented than the original request. Feel free to revert that if I made the wrong decision.

See also: this idea in #g-demand

articles/fleetctl.md Outdated Show resolved Hide resolved
articles/fleetctl.md Outdated Show resolved Hide resolved
Co-authored-by: Noah Talerman <47070608+noahtalerman@users.noreply.github.com>
Co-authored-by: Noah Talerman <47070608+noahtalerman@users.noreply.github.com>
articles/fleetctl.md Outdated Show resolved Hide resolved
articles/fleetctl.md Outdated Show resolved Hide resolved
Co-authored-by: Rachael Shaw <r@rachael.wtf>
Co-authored-by: Rachael Shaw <r@rachael.wtf>
@iansltx
Copy link
Member Author

iansltx commented Dec 12, 2024

@rachaelshaw Fixed the typo I mentioned. Should be good to go here?

@iansltx iansltx merged commit ed8c0c9 into docs-v4.61.0 Dec 12, 2024
5 checks passed
@iansltx iansltx deleted the 23910-22078-mfa-guide-updates branch December 12, 2024 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants