-
Notifications
You must be signed in to change notification settings - Fork 378
FB - Update get back access server guide #8435
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
Conversation
Hi @tcpdumpfbacke could you proceed DE + EN proof please ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fabienOVH May I inquire as to the point of these changes?
- The same info is added twice for no real reason.
- The proposed changes are not useful and on top of that will not work:
- Why would there be confusion as to the logged-in user? It it displayed in the command line at all times.
- Why would this even matter? The guide is for ppl who have lost the PW of a user account on the server and cannot log in. Why would they not know the name of the account they tried to log in with? If you were unsure about the accounts present on the system in general, use "cat /etc/passwd".
- The command "passwd" sans argument will try to change the PW of the current user. The current user is "root". This is why the guide tells you to use it specifying the username of the account you want to edit. It can be any account.
- The output of "whoami" will always be "root" at this point because you are within a root shell you have just created. The entire point is to gain elevated rights on the mounted system in order to overwrite the PW of another account. Checking which account is currently active will do nothing.
--> To improve the guide steps, remove all the edits and add the info re: "cat /etc/passwd" to see all user accounts.
Thanks for your feedbacks. I've updated the fr-fr version. Please tell me if it's ok for you. |
pages/bare_metal_cloud/dedicated_servers/replacing-user-password/guide.fr-fr.md
Outdated
Show resolved
Hide resolved
pages/bare_metal_cloud/dedicated_servers/replacing-user-password/guide.fr-fr.md
Outdated
Show resolved
Hide resolved
Thank you for your contribution @fabienOVH |
No description provided.