Skip to content
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

Токены telegram-ботов в отдельных файлах #137

Open
vvd170501 opened this issue Jan 8, 2023 · 0 comments
Open

Comments

@vvd170501
Copy link
Contributor

vvd170501 commented Jan 8, 2023

Не хватает возможности хранить токены telegram-ботов в отдельных файлах (примерно как password_file для плагина mysql).
Зачем это нужно? Например, чтобы конфиг (ejudge.xml) можно было хранить в общедоступном репозитории или использовать для шаблонной инсталляции ejudge.

Проблема: токены ботов читаются не только в самом плагине (plugins/telegram/telegram.c:prepare_func), но и в каких-то процессах сервера (lib/ejudge_cfg.c:ejudge_cfg_get_telegram_bot_id, используется в lib/serve_2.c).

Есть вопросы про ejudge_cfg_get_telegram_bot_id:

  1. Правильно ли я понимаю, что второй аргумент функции (bot_user_id) был добавлен для конфигов такого вида?
<contest>
  ...
  <telegram_bot_id>123456789</telegram_bot_id>
  ...
</contest>
  1. Почему bot_user_id - всегда NULL? Это баг, или была какая-то причина, по которой поддержка конфигов с id без токена не была добавлена?

  2. Какие значения вообще должны поддерживаться в теге <telegram_bot_id>? (на Вики он не задокументирован).
    Пусть конфиг плагина (в ejudge.xml) выглядит так:

    <plugin type="sn" name="telegram" load="yes">
      <config>
        <bots>
          <bot>123456789:abcdef</bot>
        </bots>
      </config>
    </plugin>
    • Предполагается ли поддержка токенов, отсутствующих в ejudge.xml? Т.е. contest.xml примерно такого вида:
       <contest>
         ...
         <telegram_bot_id>987654321:aabcde</telegram_bot_id>
       </contest>
    • А поддержка id ботов из ejudge.xml без токенов (как в п.1 выше)?
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

No branches or pull requests

1 participant