Алексей разработка

Задачи, обсуждения и баги

From the to-do list: Easydata.me

✔ Предусмотреть защиту от зависания скриптов рассылки СМС

Comments

Андрей Буганов on November 27, 2013:

Алексей, у нас есть риск, что:
1. Зависнет наш скрипт
2. У doForms из-за бага перестанет устанавливаться признак "Прочитано" для прочитанных анкет
Из-за этого наши клиенты могут начать получать каждые 10 минут одни и теже СМС-ки, ущерб как репутационный, так и финансовый на рассылку.
Надо добавить проверку - если по одному и тому же клиенту/проекту, т.е. одной записи на главной странице админка после 3-го срабатывания крона подряд собирается отправить 3-й раз СМС с одним и тем же текстом:
- блокировать рассылку по проекту,
- отправлять мне на +79515564857 смс с текстом: Возможно зависание скрипта информирования по проекту: [Код проекта в doForms].
Блокировку можно ставить путем снятия галочки с колонки Активность (только проверьте что-бы при снятии этой галочки не снимались галочки с телефонов в клиенте, а то менеджеру придется потом вспоминать у кого галочки стояли а у кого нет)

Желательно считать именно количество срабатываний крона, так как если мы просто поставим "если в течении 30 минут" про это можем забыть, когда например увеличим интервал между проверками наличия новых анкет

Alexey Bychkov on November 29, 2013:

Предусмотрели повторы SMSок (сделали логгирование текста отправленных SMS сообщений)

Крон пока не включали. Дадите отмашку - включим.

Alexey Bychkov on December 2, 2013:

Андрей, в срезе нашего пятничного разговора - напоминаю про крон. Дайте знать, если будет необходимость включить его.

Андрей Буганов on December 2, 2013:

Алексей, сам забыл и письмо о сообщении упустил, давай завтра с утра, меня не спрашивай сразу запускай и сообщи что запустили, еще сам напомню

Alexey Bychkov on December 3, 2013:

Андрей, включили крон!

Андрей Буганов on December 3, 2013:

Пришла СМС со списком новых анкет

Alexey Bychkov on December 3, 2013:

Всё ок?

Андрей Буганов on December 3, 2013:

В общем за то время пока крон был отключен по одному из проектов поступило уже 42 анкеты, как следствие я получил 3 смс. Когда крон будет работать постоянно это не такая проблема, но с учетом того, что:
1. Может возникнуть сбой в кроне, и он например не будет работать полдня
2. В течении 10 минут может поступить больше 20 анкет по одному проекту (маловероятно, но возможно)
3. Мы собираемся с ними интегрироваться в том числе по КГУ

возникает серьезная проблема с тем что они нормально отдают только первые 20 анкет, из-за этого приходит больше одной смс.

Алексей, напиши письмо Денису с описанием проблемы, что мы не можем получить больше 20 анкет за запрос (и почему), и то что в целом все это работает очень медленно), что-бы он переслал в doForms

Alexey Bychkov on December 3, 2013:

Написал. Тебя поставил в копию.

Андрей Буганов on December 6, 2013:

Алексей, защита срабатывает раньше чем надо. Уже второй раз так. Приходит 1 смс, а через некоторое время что скрипт завис возможно, а должно прийти 2, прежде чем заблокируется

Alexey Bychkov on December 9, 2013:

исправлено

Андрей Буганов on December 9, 2013:

ок, проверим по факту, смоделировать сложно.
Пока закрываю

Alexey Bychkov on December 12, 2013:

По поводу обновления API doForms - готово, залили, но пока работаем с бетой. Пишу им вопрос по поводу переноса этого API на production.

Андрей Буганов on December 13, 2013:

Алексей, вчера снова пришла смс о том что возможно зависание, такое ощущение что скрипт учитывает не весь текст смс, а только какую-то одну строку, вот как приходило:
21:30
Megafon_ES_20131004 - 3,
Megafon_AUDIT_0714 - 1

21:40
Megafon_ES_20131004 - 1,
Megafon-Retail_20131005 - 5,
Megafon_AUDIT_0714 - 1

Потом в 21:50 пришла СМС о блокировке скрипта, и я уверен что в тексте не отправленной смс был бы вот такой пункт (как и в предыдущих двух):
Megafon_AUDIT_0714 - 1

Alexey Bychkov on December 13, 2013:

подправли логику

Андрей Буганов on December 13, 2013:

Только что пришла СМС:
Megafon_AUDIT_0713 - 10,
Megafon_ES_20131004 - 17,
Megafon-Retail_20131005 - 2,
Megafon_AUDIT_0714 - 92

Мегафон аудит_0714 - 92 штуки, у нас же 20 максимум.

Alexey Bychkov on December 13, 2013:

сейчас же используется новый метод (писал чуть выше об этом - По поводу обновления API doForms - готово, залили, но пока работаем с бетой), поэтому дергаем все, а не по 20.

Андрей Буганов on December 13, 2013:

ок

Alexey Bychkov on December 25, 2013:

По поводу обновления API doForms - готово, залили, но пока работаем с бетой

Заменили на работу с production (doForms уведомили о выкатывании новых функций на prod)