[JIRA] (JENKINS-61379) presend scripts not run when recipient list is empty

5 views
Skip to first unread message

stefan@stemaki.dk (JIRA)

unread,
Mar 7, 2020, 8:34:02 AM3/7/20
to jenkinsc...@googlegroups.com
Stefan Rasmussen updated an issue
 
Jenkins / Improvement JENKINS-61379
presend scripts not run when recipient list is empty
Change By: Stefan Rasmussen
Issue Type: Task Improvement
Add Comment Add Comment
 
This message was sent by Atlassian Jira (v7.13.12#713012-sha1:6e07c38)
Atlassian logo

stefan@stemaki.dk (JIRA)

unread,
Mar 7, 2020, 8:34:02 AM3/7/20
to jenkinsc...@googlegroups.com
Stefan Rasmussen created an issue
 
Jenkins / Task JENKINS-61379
Issue Type: Task Task
Assignee: Alex Earl
Components: email-ext-plugin
Created: 2020-03-07 13:33
Labels: plugin
Priority: Minor Minor
Reporter: Stefan Rasmussen

When sending emails using the email-ext plugin, I'm manually populating the recipient list using the pre-send script feature. This means that the recipient list is empty at first and gets populated during the pre-send step.

Unfortunately an empty recipient list stops the entire process before the pre-send step is even run, with the message: "An attempt to send an e-mail to empty list of recipients, ignored".

It would make sense to let the pre-send script run before checking if the recipient list is empty.

slide.o.mix@gmail.com (JIRA)

unread,
Mar 7, 2020, 9:06:03 AM3/7/20
to jenkinsc...@googlegroups.com
Alex Earl commented on Improvement JENKINS-61379
 
Re: presend scripts not run when recipient list is empty

There are specific reasons that this is the ordering. You can have a dummy email address and then remove it during the pre-send script.

stefan@stemaki.dk (JIRA)

unread,
Mar 7, 2020, 10:08:03 AM3/7/20
to jenkinsc...@googlegroups.com

Hmm, not a fan of the dummy email but it will definitely work for my use case.
I didn't find any issues when testing the new order, but I trust that with a bit more time I would have eventually found it.

I'll close the issue as well as the PR on github.
Thanks for the quick response.

Reply all
Reply to author
Forward
0 new messages