This is a snapshot of Indico's old Trac site. Any information contained herein is most probably outdated. Access our new GitHub site here.

Opened 4 years ago

Closed 3 years ago

Last modified 3 years ago

#874 closed defect (fixed)

Don't let alarm tasks send mails to '<>'

Reported by: pferreir Owned by: jatrzask
Priority: high Milestone: v0.99.0
Component: General Version: 0.98-dev
Keywords: Cc:

Description (last modified by pferreir)

Otherwise they will enter the retry loop and eventually fail (silently).
We should also consider sending a warning mail to the creator of the task, in that case.

Change History (9)

comment:1 Changed 4 years ago by pferreir

  • Description modified (diff)

comment:2 Changed 3 years ago by jbenito

  • Milestone changed from v0.98.1 to v0.99.0
  • Owner changed from pferreir to jatrzask
  • Status changed from new to assigned

comment:3 Changed 3 years ago by jatrzask

  • Status changed from assigned to in_work

comment:4 Changed 3 years ago by jatrzask

  • Status changed from in_work to awaiting_merge

comment:5 Changed 3 years ago by pferreir

  • Status changed from awaiting_merge to merging

comment:6 Changed 3 years ago by Jakub Trzaskoma <jakub.piotr.trzaskoma@…>

  • Resolution set to fixed
  • Status changed from merging to closed

In 621329e2d35ed8ba7fc240bb3b4b1ef97b20b86d/indico:

[FIX] Don't let alarm tasks send mails to '<>'

  • improved 'email list' validation (now it is not possible to send email to empty list of recipients)
  • closes #874

comment:7 Changed 3 years ago by Jakub Trzaskoma <jakub.piotr.trzaskoma@…>

In f6bb4514ff46064f1231a94d3f3dcebbbb7e33d3/indico:

[FIX] Filtering empty mail addresses ('<>') just before sending a mail

  • added filtering empty mail addresses ('<>') from recipients and CC list
  • added checking if recipients list is not empty ('<>')
  • closes #874

comment:8 Changed 3 years ago by Jakub Trzaskoma <jakub.piotr.trzaskoma@…>

In 9c05efa5caccf20b8b0f6940739a0398b650c3bb/indico:

[FIX] Don't let alarm tasks send mails to '<>'

  • improved 'email list' validation (now it is not possible to send email to empty list of recipients)
  • closes #874

comment:9 Changed 3 years ago by Jakub Trzaskoma <jakub.piotr.trzaskoma@…>

In 4cd6ae22c517786233036b0d166b92d07834c279/indico:

[FIX] Filtering empty mail addresses ('<>') just before sending a mail

  • added filtering empty mail addresses ('<>') from recipients and CC list
  • added checking if recipients list is not empty ('<>')
  • closes #874
Note: See TracTickets for help on using tickets.