[PR #17605] service.bat: !ServiceName! выводится как есть при запуске стратегии #500

New Issue

Open

opened 2026-09-21 11:18:13 +03:00 by mirror-bot · 0 comments

mirror-bot commented 2026-09-21 11:18:13 +03:00

Copy link

Copy source

📋 Pull Request Information

Original PR: https://github.com/Flowseal/zapret-discord-youtube/pull/17605
Author: @klaymm
Created: 9/20/2026
Status: 🔄 Open

Base: main ← Head: fix/servicename

---

📝 Commits (1)

• 163e24a service.bat: !ServiceName! выводится как есть при запуске стратегии

📊 Changes

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

View changed files

📝 service.bat (+1 -1)

📄 Description

В :test_service сообщение о STOP_PENDING собрано через отложенное расширение:

call :PrintYellow "!ServiceName! is STOP_PENDING, that may be caused by a conflict with another bypass. Run Diagnostics to try to fix conflicts"

Но ветка status_zapret отрабатывает в самом начале файла, до setlocal EnableDelayedExpansion. А зовёт её каждый general*.bat первой строкой:

call service.bat status_zapret

В результате при запуске любой стратегии, если служба висит в STOP_PENDING, пользователь видит !ServiceName! вместо имени службы. Через меню service.bat то же сообщение выводится правильно, поэтому баг легко пропустить.

Репро:

@echo off
if "%~1"=="status_zapret" (
    call :test_service zapret soft
    exit /b
)
setlocal EnableDelayedExpansion
call :test_service zapret soft
exit /b

:test_service
set "ServiceName=%~1"
echo OLD: !ServiceName! is STOP_PENDING
echo NEW: %ServiceName% is STOP_PENDING
exit /b

> repro.bat status_zapret     (как вызывает general*.bat)
OLD: !ServiceName! is STOP_PENDING
NEW: zapret is STOP_PENDING

> repro.bat                   (как вызывает меню)
OLD: zapret is STOP_PENDING
NEW: zapret is STOP_PENDING

Заменил на %ServiceName% - так же, как уже сделано строкой выше в этой же подпрограмме. Работает в обоих путях вызова.

Как по мне, само сообщение важное, оно советует запустить диагностику, поэтому читаемым оно быть должно.

---

_{🔄 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/17605 **Author:** [@klaymm](https://github.com/klaymm) **Created:** 9/20/2026 **Status:** 🔄 Open **Base:** `main` ← **Head:** `fix/servicename` --- ### 📝 Commits (1) - [`163e24a`](https://github.com/Flowseal/zapret-discord-youtube/commit/163e24a126899b0e23dd83f949ff969bb95f314b) service.bat: !ServiceName! выводится как есть при запуске стратегии ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `service.bat` (+1 -1) </details> ### 📄 Description # В `:test_service` сообщение о `STOP_PENDING` собрано через отложенное расширение: ```bat call :PrintYellow "!ServiceName! is STOP_PENDING, that may be caused by a conflict with another bypass. Run Diagnostics to try to fix conflicts" ``` Но ветка `status_zapret` отрабатывает в самом начале файла, до `setlocal EnableDelayedExpansion`. А зовёт её каждый `general*.bat` первой строкой: ```bat call service.bat status_zapret ``` В результате при запуске любой стратегии, если служба висит в `STOP_PENDING`, пользователь видит `!ServiceName!` вместо имени службы. Через меню `service.bat` то же сообщение выводится правильно, поэтому баг легко пропустить. Репро: ```bat @echo off if "%~1"=="status_zapret" ( call :test_service zapret soft exit /b ) setlocal EnableDelayedExpansion call :test_service zapret soft exit /b :test_service set "ServiceName=%~1" echo OLD: !ServiceName! is STOP_PENDING echo NEW: %ServiceName% is STOP_PENDING exit /b ``` ``` > repro.bat status_zapret (как вызывает general*.bat) OLD: !ServiceName! is STOP_PENDING NEW: zapret is STOP_PENDING > repro.bat (как вызывает меню) OLD: zapret is STOP_PENDING NEW: zapret is STOP_PENDING ``` Заменил на `%ServiceName%` - так же, как уже сделано строкой выше в этой же подпрограмме. Работает в обоих путях вызова. Как по мне, само сообщение важное, оно советует запустить диагностику, поэтому читаемым оно быть должно. --- _{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

mirror-bot added the pull-request label 2026-09-21 11:18:13 +03:00

Sign in to join this conversation.

No Branch/Tag Specified

Branches Tags

main

1.10.4

1.10.3

1.10.2

1.10.1

1.10.0

1.9.9d

1.9.9c

1.9.9b

1.9.9a

1.9.8c

1.9.8b

1.9.8

1.9.7b

1.9.7

1.9.6

1.9.5

1.9.4

1.9.3

1.9.2

1.9.1

1.9.0b

1.9.0

1.8.5

1.8.4

1.8.3

1.8.2

1.8.1

1.8.0

1.7.2b

1.7.2

1.7.1

1.7.0

1.6.6

1.6.5

1.6.4

1.6.3

1.6.2

1.6.1

1.6.0

1.5.2

1.5.1

1.5.0

1.4.1

1.4.0

1.3.0

1.2.0

1.1.1

1.1.0

1.0.0

Labels

Clear labels

Stale

WARNING

discussion

duplicate

Auto-created by moderation bot

faq-candidate

games

in-progress

incident

pinned

pull-request

Mirrored from GitHub Pull Request

Обход

No labels pull-request

Milestone

No items

No Milestone

Assignees

Clear assignees

Fiden MrCat

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Utilities/zapret-discord-youtube#500