✔ Предусмотреть защиту от зависания скриптов рассылки СМС
(Completed by Андрей Буганов on December 9, 2013)
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: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)
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:
1. Может возникнуть сбой в кроне, и он например не будет работать полдня
2. В течении 10 минут может поступить больше 20 анкет по одному проекту (маловероятно, но возможно)
3. Мы собираемся с ними интегрироваться в том числе по КГУ
возникает серьезная проблема с тем что они нормально отдают только первые 20 анкет, из-за этого приходит больше одной смс.
Алексей, напиши письмо Денису с описанием проблемы, что мы не можем получить больше 20 анкет за запрос (и почему), и то что в целом все это работает очень медленно), что-бы он переслал в doForms
Alexey Bychkov on December 3, 2013:
Андрей Буганов on December 6, 2013:
Alexey Bychkov on December 9, 2013:
Андрей Буганов on December 9, 2013:
Пока закрываю
Alexey Bychkov on December 12, 2013:
Андрей Буганов 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:
Андрей Буганов on December 13, 2013:
Alexey Bychkov on December 25, 2013:
Заменили на работу с production (doForms уведомили о выкатывании новых функций на prod)