Skip to content
This repository has been archived by the owner on Jun 1, 2023. It is now read-only.

Update securing-your-node.md 2FA instructions #209

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update securing-your-node.md 2FA instructions #209

wants to merge 1 commit into from

Conversation

IslamAbdelRahman
Copy link
Contributor

I was not able to enable 2FA for a long time until I finally figured out that I need to set PermitRootLogin to yes

@IslamAbdelRahman
Update securing-your-node.md 2FA instructions
c841f20 
I was not able to enable 2FA for a long time until I finally figured out that I need to set `PermitRootLogin` to `yes`
@ryanscovill
Copy link
Contributor

Would not recommend enabling PermitRootLogin. Something else must be configured incorrectly because root login is not required for 2FA.

@IslamAbdelRahman
Copy link
Contributor Author

@ryanscovill this is the only way I was able to solve the problem by follwing this guide
https://serverfault.com/questions/869627/google-authenticator-pam-on-ssh-blocks-root-login-without-2fa

Let me know if there is something else I could have done

@ryanscovill
Copy link
Contributor

@IslamAbdelRahman I have followed the doc 2FA guide without issue using a non-root user to login. This workaround to allow root login seems specific to your specific user configuration and not a general solution for everyone.

@jclapis
Copy link
Member

jclapis commented May 3, 2023

@actuallymentor would you mind taking a look here?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@IslamAbdelRahman @ryanscovill @jclapis