Commit 2714cb1e authored by Baoqi's avatar Baoqi
Browse files

before send mail, check whether no receivers && cc receivers. 

to avoid exception javax.mail.SendFailedException: No recipient
addresses. Especially from SQL Task
parent 9c79a0f9

escheduler-alert/src/main/java/cn/escheduler/alert/utils/MailUtils.java

+7 −2
Original line number Diff line number Diff line
@@ -102,11 +102,16 @@ public class MailUtils { 
     */

    public static Map<String,Object> sendMails(Collection<String> receivers, Collection<String> receiversCc, String title, String content, ShowType showType) {

        Map<String,Object> retMap = new HashMap<>();

        retMap.put(Constants.STATUS, false);





        receivers.removeIf((from) -> (StringUtils.isEmpty(from)));



        // if there is no receivers && no receiversCc, no need to process

        if (CollectionUtils.isEmpty(receivers) && CollectionUtils.isEmpty(receiversCc)) {

            retMap.put(Constants.STATUS, true);

            return retMap;

        }



        retMap.put(Constants.STATUS, false);

        if (showType == ShowType.TABLE || showType == ShowType.TEXT){

            // send email

            HtmlEmail email = new HtmlEmail();