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

Add MFA enforcement on popular gems blog post #121

Merged
merged 7 commits into from
Aug 16, 2022

Conversation

jenshenny
Copy link
Member

@jenshenny jenshenny commented Aug 2, 2022

Part of rubygems/rubygems.org#3163
Following up from the Phase 2 blog post: #110

This post will announce the MFA enforcement policy to the community. Some key points that are addressed

  • The release date (August 15)
  • Who is affected (gem maintainers with a gem >180 million downloads)
  • What does enforcement look like (unable to access profile pages, unable to gem signin, push, yank, owners -a/-r)
  • Future plans coming out soon
  • Working on WebAuthn

@jenshenny jenshenny marked this pull request as ready for review August 2, 2022 19:19
Comment on lines 7 to 9
<p align="center">
<img src="/images/gem-with-mfa-flag-dropshadow.png" alt="Doodle of a RubyGem wearing an MFA hat, holding a flag with a checkmark" width="300"/>
</p>
Copy link
Member Author

Choose a reason for hiding this comment

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

Used the same image as the last post. Do we want to change up the doodle this time around to keep it interesting?

Copy link
Member

Choose a reason for hiding this comment

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

If you have time to update it, sure. Using the same if also fine.

Copy link
Member Author

Choose a reason for hiding this comment

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

@bettymakes quickly stirred up another doodle 🚀 , replaced the OG doodle with that one.

_posts/2022-08-15-requiring-mfa-on-popular-gems.md Outdated Show resolved Hide resolved
Comment on lines 7 to 9
<p align="center">
<img src="/images/gem-with-mfa-flag-dropshadow.png" alt="Doodle of a RubyGem wearing an MFA hat, holding a flag with a checkmark" width="300"/>
</p>
Copy link
Member

Choose a reason for hiding this comment

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

If you have time to update it, sure. Using the same if also fine.

_posts/2022-08-15-requiring-mfa-on-popular-gems.md Outdated Show resolved Hide resolved
Copy link
Contributor

@bettymakes bettymakes left a comment

Choose a reason for hiding this comment

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

✅ This looks great Jenny! Thanks for working on it. I have two small comments, one is purely a stylistic choice.


Two months ago, we outlined our [commitment](https://blog.rubygems.org/2022/06/13/making-packages-more-secure.html) to making Ruby’s supply chain more secure. To combat account takeovers — the second most common software supply chain attack — we announced a policy to require MFA on at least the top-100 RubyGems packages.

Today (August 15th, 2022), we will begin to enforce MFA on owners of gems with over 180 million total downloads. Users in this category who do not have MFA enabled on the `UI and API` or `UI and gem signin` level will not be able to edit their profile on the web, perform [privileged actions](https://guides.rubygems.org/mfa-requirement-opt-in/#privileged-operations) (i.e. push and yank gems, or add and remove gem owners) or sign in on the command line until they [configure MFA](https://guides.rubygems.org/setting-up-multifactor-authentication/).
Copy link
Contributor

Choose a reason for hiding this comment

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

Super tiny nit: It's just a stylistic choice. I prefer the Oxford comma. Its proper usage would be when there is a list of three or more things, which applies to this case.

That said, with or without the comma, it's correct.

Users in this category who do not have MFA enabled on the UI and API or UI and gem signin level will not be able to edit their profile on the web, perform privileged actions (i.e. push and yank gems, or add and remove gem owners),👈 or sign in on the command line until they configure MFA.

<img src="/images/gem-with-thumbs-up-mfa-dropshadow.png" alt="Doodle of a RubyGem wearing a MFA hat, giving a thumbs up" width="300"/>
</p>

Two months ago, we outlined our [commitment](https://blog.rubygems.org/2022/06/13/making-packages-more-secure.html) to making Ruby’s supply chain more secure. To combat account takeovers — the second most common software supply chain attack — we announced a policy to require MFA on at least the top-100 RubyGems packages.
Copy link
Contributor

Choose a reason for hiding this comment

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

We should spell out "MFA" and define the acronym before using the acronym elsewhere.

Two months ago, we outlined our commitment to making Ruby’s supply chain more secure. To combat account takeovers — the second most common software supply chain attack — we announced a policy to require 👉 multi-factor authentication (MFA) 👈 on at least the top-100 RubyGems packages.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes agreed, great catch! 🚀

@indirect
Copy link
Member

Looking good, thanks all.

@indirect indirect merged commit 91be544 into rubygems:master Aug 16, 2022
@bettymakes bettymakes deleted the mfa-required-release branch August 16, 2022 03:47
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.

4 participants