Skip to content

Feature/opsec/mfa#452

Merged
mattaereal merged 4 commits intosecurity-alliance:developfrom
shallem:feature/opsec/mfa
Apr 9, 2026
Merged

Feature/opsec/mfa#452
mattaereal merged 4 commits intosecurity-alliance:developfrom
shallem:feature/opsec/mfa

Conversation

@shallem
Copy link
Copy Markdown
Contributor

@shallem shallem commented Apr 8, 2026

  • I added a new overview of MFA best practices, and I also updated the contributor file
  • Fixed build issues in contributors.json from my first PR
  • @mattaereal please take a look

@shallem shallem force-pushed the feature/opsec/mfa branch from 2e96f2b to bb5535c Compare April 8, 2026 17:34
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 8, 2026

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
frameworks ✅ Ready (View Log) Visit Preview 6f2e07e

@mattaereal mattaereal self-assigned this Apr 8, 2026
@mattaereal mattaereal added the content:add This issue or PR adds content or suggests to label Apr 8, 2026
Revised language for clarity and emphasis on MFA importance. Updated recommendations for MFA methods and highlighted security considerations for passkeys.
@mattaereal
Copy link
Copy Markdown
Collaborator

I added a suggestion and clarification right towards the end. You could create a separate page for each bullet, to go a little bit deeper into each of them, and explaining why, plus suggesting the best practices for each

Copy link
Copy Markdown
Collaborator

@scode2277 scode2277 left a comment

Choose a reason for hiding this comment

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

gtg @mattaereal!

I agree on expanding the general recommendations into new pages or even making this one bigger, but i think we can make it in another PR so we can ship and iterate on an already pushed page

@shallem
Copy link
Copy Markdown
Contributor Author

shallem commented Apr 8, 2026

Just pushed another commit to address @mattaereal 's feedback. Please review.

@mattaereal mattaereal merged commit 04dc3e0 into security-alliance:develop Apr 9, 2026
3 checks passed
@mattaereal
Copy link
Copy Markdown
Collaborator

Congrats on your first collab! We can come back later to improve this section once we have more context with the others. We'll need to adapt the three long paragraphs into something separate like : Passkey considerations, a few bullets, and a note, but so far so good. As I said, I rather have the information there that nothing at all, since it's easier to work with something we have!

@shallem
Copy link
Copy Markdown
Contributor Author

shallem commented Apr 9, 2026

Agreed 100% @mattaereal . My next big content push will be around password security, and in that context I will revisit the entire MFA section. I need to educate myself how to split content into different pages, and that will help me divide and streamline. Coming soon.

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

Labels

content:add This issue or PR adds content or suggests to

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants