Skip to content

Verifying the server user to work with sudo #1254

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

Merged
merged 31 commits into from
Dec 1, 2024

Conversation

lunardunno
Copy link
Collaborator

@lunardunno lunardunno commented Nov 16, 2024

Changing the mechanism for checking that the server user to work with sudo

Changing the mechanism for checking that the username is root
Checking if the user is included in the wheel group
* Checking requirements in script

Checking requirements for sudo users in script

* Adding error handling

Adding error handling in the server controller for:
Sudo package is not pre-installed for sudo users.
Server user or associated group is not listed in the sudoers file.
Server user password required

* adding error codes

* added extended error descriptions
Сhecking sudo permission for root.
Сhanging the verification method 
and redefining the system language.
Сommand to use home directory name if whoami returns error or is missing.
Сommand to use home directory name if whoami returns error or is missing.
One of the errors has been renamed.
Fixed detection text in StdOut.
Correction of the text of the extended description of the package manager error
Correction of the text of the extended description of the package manager error
@lunardunno lunardunno changed the title checking that the username is root Verifying the server user to work with sudo Nov 20, 2024
@lunardunno lunardunno changed the base branch from dev to checking_sudo_permissions December 1, 2024 08:24
@lunardunno lunardunno marked this pull request as ready for review December 1, 2024 08:28
@lunardunno lunardunno merged commit 076b076 into checking_sudo_permissions Dec 1, 2024
7 checks passed
@pokamest pokamest deleted the sudo_permission_check branch January 6, 2025 15:40
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.

1 participant