georgiydemo / fatimetable Goto Github PK

0.0 0.0 0.0 369 KB

SMS-оповещения о расписании пар с ИОП. Не актуально т.к. расписание теперь на РУЗ

License: GNU General Public License v3.0

Python 95.63% Dockerfile 3.57% Shell 0.80%

docker docker-compose fa-university fifo redis sms

fatimetable's Introduction

    /\_/\
   ( o.o )
   >^<

fatimetable's People

Contributors

Watchers

fatimetable's Issues

Понять на каком уровне вводить sleep для того, чтоб не забанил оператор СМС

На уровне python-rq или внутри sender'а?
rq/rq#526

Конфиги в YAML

Все конфиги (URL, таймеры и т.д.) вынести в единый файл settings.yaml и успокоиться

Клиент в виде бота в Telegram

Также будет крутиться-вертеться в docker в сервером и т д

Организация затяжного делея

БОльший промежуток времени рассылки
Рассылаем 30 сообщений - ждём и т.д.

Перерисовать диаграмму архитектуры в README.md

Добавить отдельно на диаграмму модуль async в Flask + удаление клиента и замена на Telegram

Более адекватная проверка на то, что нет пар

Когда нет пар - {}
Тогда даже не надо вызывать модуль обработки

Обновить либы в req.txt во всех контейнерах

Т.к. тот же requests уже нормально так устарел

Кастомная отправка сообщения всем абонентам системы

Методы на стороне server (Flask API) и на стороне telegram-cli для моментальной рассылки кастомного сообщения абсолютно всем абонентам в базе

Организация алфавита замены

Необходим для того, чтоб не было дубляжа сообщений
Пример:
А - A
O - О и т. д.

Переход от GET к POST

И там теряется знак "+" в ноере телефона от клиента до сервера по методу add_number

Использование другого метода для возврата расписания на FA API

Т.к. Текущий метод fa_api.FaClass.get_timetable_byday не позволяет указать id группы для её расписания (или я что-то не понял)

В любом случае, надо это еще раз посмотреть

Обновление токена авторизации

Сделать отдельный async с автообновлением сессии авторизации каждые M минут/часов

Вход в ИОП перед очередью FIFO, а не при инициализации сервера

Ващ потом сделать async по #7

Глобальный переезд в Docker

Перенос всех частей в docker и запуск уже непосредственно на боевом сервере

Рандомный time.sleep от 25 до 35

При отправке SMS на Sender'е сейчас фиксированный интервал из конфига. Желательно сделать его в каком-либо диапазоне в целях относительного нарушения систематизации рассылки.

FAtimetable/docker/sender/sender.py

Line 36 in 9f270a5

time.sleep(self.config["SMS_TIME_SLEEP"])

Проверка на то, что нет пар

Если пар нет в этот день, то оповещать не надо
Сейчас никакой проверки на это неть

Логирование

Сделать чтение логов с docker-контейнеров, возможный вынос в клиент телеграма

Какие-либо полезные действия со стороны RQExceptionHandler

Обработчик ошибок RQExceptionHandler пока просто простаивает, было бы неплохо взять и сделать из него что-то полезное
Оповещение? Логирование? SentryIO?

Асинхронная генерация расписалова на стороне core, а не sender

Сейчас происходит генерация в sender для каждого отдельного пользователя, что в корне неправильно и тупо (БД нам зачем?))

Пусть будет перегенериться раз в N промежуток времени на стороне core с и будет заноситься в таблицу №4 и все будут счастливы

Контейнер RQ DASHBOARD

Организовать в docker-compose контейнер с dashboard для визуализации очереди

Асинхронная генерация расписание на стороне async_sender

Сейчас происходит получение расписания на стороне async_sender только когда пришло время его отправлять (вечером в 18-19 часов) и запись его в Redis, чтоб не DDosить ИОП.

На самом деле это не совсем правильно:
Проще генерировать расписание в async-режиме и заносить в БД обновления (скажем раз в N дней) и потом уже в sender'е спокойно получать только значения из БД

В прасинге JSON -> string проблемы

Необходимо сделать биг дату с расписаниями и тестить по ним

[CRITICAL] RQ Worker конфликтует с Редиской

redis.exceptions.ReadOnlyError: Command # 1 (DEL rq:worker:57604ccde8a74fbea8e0ca5dc9ef8897) of pipeline caused error: Command # 1 (DEL rq:worker:57604ccde8a74fbea8e0ca5dc9ef8897) of pipeline caused error: You can't write against a read only replica.