[PR #1254] [MERGED] Verifying the server user to work with sudo #2287

New issue

Closed

opened 2026-10-13 21:20:32 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-13 21:20:32 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/amnezia-vpn/amnezia-client/pull/1254
Author: @lunardunno
Created: 11/16/2024
Status: ✅ Merged
Merged: 12/1/2024
Merged by: @lunardunno

Base: checking_sudo_permissions ← Head: sudo_permission_check

📝 Commits (10+)

3d95b14 checking that the username is root
9695a5b wheel group check (#1198)
0de55d8 Checking requirements in script (#1210)
3dd2fad checking sudo permission for root
5890e94 Username if whoami returns an error
e1bd7a4 Last line correction
d3f8668 Last line correction
6b18bc1 Last line correction
2af3d27 corrections
9a2ae00 One of the errors has been renamed

📊 Changes

13 files changed (+48 additions, -38 deletions)

View changed files

📝 client/core/controllers/serverController.cpp (+6 -4)
📝 client/core/defs.h (+3 -0)
📝 client/core/errorstrings.cpp (+5 -2)
📝 client/server_scripts/check_user_in_sudo.sh (+4 -2)
📝 client/server_scripts/prepare_host.sh (+1 -1)
📝 client/translations/amneziavpn_ar_EG.ts (+3 -3)
📝 client/translations/amneziavpn_fa_IR.ts (+4 -4)
📝 client/translations/amneziavpn_hi_IN.ts (+4 -4)
📝 client/translations/amneziavpn_my_MM.ts (+4 -4)
📝 client/translations/amneziavpn_ru_RU.ts (+3 -3)
📝 client/translations/amneziavpn_uk_UA.ts (+4 -4)
📝 client/translations/amneziavpn_ur_PK.ts (+3 -3)
📝 client/translations/amneziavpn_zh_CN.ts (+4 -4)

📄 Description

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

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/amnezia-vpn/amnezia-client/pull/1254 **Author:** [@lunardunno](https://github.com/lunardunno) **Created:** 11/16/2024 **Status:** ✅ Merged **Merged:** 12/1/2024 **Merged by:** [@lunardunno](https://github.com/lunardunno) **Base:** `checking_sudo_permissions` ← **Head:** `sudo_permission_check` --- ### 📝 Commits (10+) - [`3d95b14`](https://github.com/amnezia-vpn/amnezia-client/commit/3d95b14fc145164ae5c37f29e59611ed019cd073) checking that the username is root - [`9695a5b`](https://github.com/amnezia-vpn/amnezia-client/commit/9695a5b01523ca8db420a16f0d7f2f38bf805bf5) wheel group check (#1198) - [`0de55d8`](https://github.com/amnezia-vpn/amnezia-client/commit/0de55d84c9d32a6905448285f5d4bf8b4889c9ea) Checking requirements in script (#1210) - [`3dd2fad`](https://github.com/amnezia-vpn/amnezia-client/commit/3dd2fad6aa8c56b7b46a8f2c1e3687ff9f58dad2) checking sudo permission for root - [`5890e94`](https://github.com/amnezia-vpn/amnezia-client/commit/5890e942471242d8f9c31a8fcb72e4f838f1dabb) Username if whoami returns an error - [`e1bd7a4`](https://github.com/amnezia-vpn/amnezia-client/commit/e1bd7a42cb4e78e26a3e9da230134a5d13ff61d4) Last line correction - [`d3f8668`](https://github.com/amnezia-vpn/amnezia-client/commit/d3f866889ad354d17763ca8c2a7991cb801d1493) Last line correction - [`6b18bc1`](https://github.com/amnezia-vpn/amnezia-client/commit/6b18bc15329132c334b76a509254aa12365cec48) Last line correction - [`2af3d27`](https://github.com/amnezia-vpn/amnezia-client/commit/2af3d2799a2d58d7488199ebb22ea759ce0eca7a) corrections - [`9a2ae00`](https://github.com/amnezia-vpn/amnezia-client/commit/9a2ae0047eb4d62a3ed7fea6ae6b1066ddeadb84) One of the errors has been renamed ### 📊 Changes **13 files changed** (+48 additions, -38 deletions) <details> <summary>View changed files</summary> 📝 `client/core/controllers/serverController.cpp` (+6 -4) 📝 `client/core/defs.h` (+3 -0) 📝 `client/core/errorstrings.cpp` (+5 -2) 📝 `client/server_scripts/check_user_in_sudo.sh` (+4 -2) 📝 `client/server_scripts/prepare_host.sh` (+1 -1) 📝 `client/translations/amneziavpn_ar_EG.ts` (+3 -3) 📝 `client/translations/amneziavpn_fa_IR.ts` (+4 -4) 📝 `client/translations/amneziavpn_hi_IN.ts` (+4 -4) 📝 `client/translations/amneziavpn_my_MM.ts` (+4 -4) 📝 `client/translations/amneziavpn_ru_RU.ts` (+3 -3) 📝 `client/translations/amneziavpn_uk_UA.ts` (+4 -4) 📝 `client/translations/amneziavpn_ur_PK.ts` (+3 -3) 📝 `client/translations/amneziavpn_zh_CN.ts` (+4 -4) </details> ### 📄 Description Changing the mechanism for checking that the server user to work with sudo --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-13 21:20:32 +00:00