[PR #17124] [CLOSED] Validate service manager startup and return errors #481

Closed
opened 2026-08-29 05:22:42 +03:00 by mirror-bot · 0 comments

📋 Pull Request Information

Original PR: https://github.com/Flowseal/zapret-discord-youtube/pull/17124
Author: @anntr1k3
Created: 8/28/2026
Status: ❌ Closed

Base: main ← Head: fix/validate-service-startup


📝 Commits (1)

  • 23d1972 Validate service manager startup

📊 Changes

1 file changed (+51 additions, -7 deletions)

View changed files

📝 service.bat (+51 -7)

📄 Description

Summary

Validate actual administrative elevation and propagate failures from required startup operations.

Changes

  • verify the admin token instead of trusting the admin argument;
  • check required commands and extracted directories;
  • propagate user-list creation failures;
  • quote command lookup;
  • use safer elevation argument construction and return codes.

Validated locally: a non-elevated admin invocation exits before the menu, and existing user lists remain unchanged.


🔄 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/Flowseal/zapret-discord-youtube/pull/17124 **Author:** [@anntr1k3](https://github.com/anntr1k3) **Created:** 8/28/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `fix/validate-service-startup` --- ### 📝 Commits (1) - [`23d1972`](https://github.com/Flowseal/zapret-discord-youtube/commit/23d1972e6a2885f050066a9ee1f4e913ab712141) Validate service manager startup ### 📊 Changes **1 file changed** (+51 additions, -7 deletions) <details> <summary>View changed files</summary> 📝 `service.bat` (+51 -7) </details> ### 📄 Description ## Summary Validate actual administrative elevation and propagate failures from required startup operations. ## Changes - verify the admin token instead of trusting the admin argument; - check required commands and extracted directories; - propagate user-list creation failures; - quote command lookup; - use safer elevation argument construction and return codes. Validated locally: a non-elevated admin invocation exits before the menu, and existing user lists remain unchanged. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
mirror-bot added the pull-request label 2026-08-29 05:22:42 +03:00
mirror-bot changed title from [PR #17124] Validate service manager startup and return errors to [PR #17124] [CLOSED] Validate service manager startup and return errors 2026-08-29 21:22:15 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Utilities/zapret-discord-youtube#481