{"action":"opened","issue":{"url":"https://api.github.com/repos/civicrm/org.civicrm.flexmailer/issues/23","repository_url":"https://api.github.com/repos/civicrm/org.civicrm.flexmailer","labels_url":"https://api.github.com/repos/civicrm/org.civicrm.flexmailer/issues/23/labels{/name}","comments_url":"https://api.github.com/repos/civicrm/org.civicrm.flexmailer/issues/23/comments","events_url":"https://api.github.com/repos/civicrm/org.civicrm.flexmailer/issues/23/events","html_url":"https://github.com/civicrm/org.civicrm.flexmailer/issues/23","id":335355821,"node_id":"MDU6SXNzdWUzMzUzNTU4MjE=","number":23,"title":"Parsing SMTP Socket Errors","user":{"login":"pbatroff","id":22370179,"node_id":"MDQ6VXNlcjIyMzcwMTc5","avatar_url":"https://avatars1.githubusercontent.com/u/22370179?v=4","gravatar_id":"","url":"https://api.github.com/users/pbatroff","html_url":"https://github.com/pbatroff","followers_url":"https://api.github.com/users/pbatroff/followers","following_url":"https://api.github.com/users/pbatroff/following{/other_user}","gists_url":"https://api.github.com/users/pbatroff/gists{/gist_id}","starred_url":"https://api.github.com/users/pbatroff/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pbatroff/subscriptions","organizations_url":"https://api.github.com/users/pbatroff/orgs","repos_url":"https://api.github.com/users/pbatroff/repos","events_url":"https://api.github.com/users/pbatroff/events{/privacy}","received_events_url":"https://api.github.com/users/pbatroff/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-06-25T10:53:03Z","updated_at":"2018-06-25T10:53:03Z","closed_at":null,"author_association":"NONE","body":"Apparently syntactically valid emails with invalid domains aren't processed properly, and if there are more than 5 such mails, the whole cron-job is interrupted, indefinitely. Probably even other jobs executed after the Mail job (though I haven't verified this yet).\r\n\r\nIn this case the whole mailing system is blocked, since old mailings wont be completed and the cronjob fails.\r\n\r\nIn the [code](https://github.com/civicrm/org.civicrm.flexmailer/blob/master/src/Listener/DefaultSender.php#L103) it seems like after 5 smtp errors the process is interrupted. \r\n\r\nI wonder if this is the correct behavior in this specific case! While I agree that interrupting the process in case of connection problem is correct, invalid domain should lead to setting the email address on hold for the contact. \r\n\r\nIs it possible, or would it make sense to include some form of feedback parsing at this point?"}}
{"actor":{"display_login":"pbatroff"}}