On Tue, Oct 24, 2017 at 4:37 AM, Brian Stinson <brian@bstinson.com> wrote:
On Oct 23 09:34, Pavel Kajaba wrote:
> Hello again,
>
> Do you have any suggestion how to tackle this issue?
>
> I will be really thankful!
>
> On Fri, Oct 20, 2017 at 5:35 PM Pavel Kajaba <pavel@redhat.com> wrote:
>
> > Hello,
> >
> > I would like to ask how do you are you implementing $SUBJECT in your
> > project?
> >
> > I tried to setup email-ext plugin, but there was CVE [1], which disabled
> > option to send emails to a user which are not registered in Jenkins.
> >
> > There is a workaround to shift this behavior, but I am not sure whether
> > you want to enable it:
> > -Dhudson.tasks.MailSender.SEND_TO_UNKNOWN_USERS=true
> >
> > So my question is, what would be the best solution to implement
> > notifications in current ci.centos.org Jenkins instance?
> >
> > Thanks,
> > Pavel Kajaba
> >
> > [1]
> > https://jenkins.io/security/advisory/2017-03-20/#emails-were-sent-to-addresses-not-associated-with-actual-users-of-jenkins-by-mailer-plugin-and-email-extension-plugin
> >
> >

> _______________________________________________
> Ci-users mailing list
> Ci-users@centos.org
> https://lists.centos.org/mailman/listinfo/ci-users


If you have a mailing list or person you'd like to notify based on your
jobs, let me know and we can work on this.

Or can you point at the config that you used to configure your notification?

Current configuration can be found here: https://github.com/almighty/almighty-jobs/blob/master/devtools-ci-index.yaml#L550 
I am able to send emails to specifed users. That's quite simple. However, I want to send emails to users who reviewed (merged) and created pull request. 
I have it configured, but I am receiving warning at the end of the logs as you can see here: https://ci.centos.org/job/devtools-f8a-master-deploy-e2e-test/258/console

Thanks!
 

--Brian