Skip to content

feat: add alarm channels INT-939#199

Draft
victor-fedorov-wb wants to merge 4 commits into
masterfrom
feature/add-alarm-channels-INT-939
Draft

feat: add alarm channels INT-939#199
victor-fedorov-wb wants to merge 4 commits into
masterfrom
feature/add-alarm-channels-INT-939

Conversation

@victor-fedorov-wb
Copy link
Copy Markdown

@victor-fedorov-wb victor-fedorov-wb commented May 8, 2026


Что происходит; кому и зачем нужно:

Добавляет в сервис алармов три новых типа получателей: ВК-сообщество,
бот MAX и произвольный HTTP-вебхук (Discord/Slack/ntfy/внутренний API
существующие alarms.conf не затронуты.

  • Notify.sendWebhook(opts) в scripts/lib.js — универсальный транспорт поверх curl.
  • Три новых записи в Alarms.recipientTypes, все через Notify.sendWebhook:
    • webhookurl, method, headers, contentType, bodyTemplate (подстановка {}/{{ expr }} как в
      email.subject).
    • vkapi.vk.com/method/messages.send, поля token, peerId, опц. apiVersion.
    • maxplatform-api.max.ru/messages, поля token, chatId.
  • Схемы (alarms.schema.json и alarms-restricted.schema.json) дополнены тремя ветками oneOf + русские переводы.
  • README — описание новой функции и пример конфига для каждого типа.

Заметка

bodyTemplate подставляет {} без JSON-эскейпа. Для статичных alarmMessage норм, но при динамическом тексте с кавычками — JSON-тело может сломаться.


Что поменялось для пользователей:
image image
image


Как проверял/а:

@codacy-production
Copy link
Copy Markdown

codacy-production Bot commented May 8, 2026

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 126 complexity · 0 duplication

Metric Results
Complexity 126
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

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