Skip to content

Commit 735ee27

Browse files
authored
doc: remove mention of SMS 2FA
PR-URL: #60707 Refs: nodejs/admin#1005 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Ulises Gascón <[email protected]>
1 parent c4ca7d6 commit 735ee27

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

onboarding.md

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,8 @@ onboarding session.
88
* If the new Collaborator is not yet a member of the nodejs GitHub organization,
99
confirm that they are using [two-factor authentication][]. It will not be
1010
possible to add them to the organization if they are not using two-factor
11-
authentication. If they cannot receive SMS messages from GitHub, try
12-
[using a TOTP mobile app][].
11+
authentication. Ask them to use secure two-factor methods: authenticator apps,
12+
passkeys, security keys, and/or the GitHub mobile app.
1313
* Suggest the new Collaborator install [`@node-core/utils`][] and
1414
[set up the credentials][] for it.
1515

@@ -303,5 +303,4 @@ needs to be pointed out separately during the onboarding.
303303
[set up the credentials]: https://github.com/nodejs/node-core-utils#setting-up-github-credentials
304304
[static-analysis]: doc/contributing/static-analysis.md
305305
[two-factor authentication]: https://help.github.com/articles/securing-your-account-with-two-factor-authentication-2fa/
306-
[using a TOTP mobile app]: https://help.github.com/articles/configuring-two-factor-authentication-via-a-totp-mobile-app/
307306
[who-to-cc]: doc/contributing/collaborator-guide.md#who-to-cc-in-the-issue-tracker

0 commit comments

Comments
 (0)