| The method notifyGroupOfTask of the class BasicWorkflowService has a try catch the outside the for loop that iterates through ePersons. It catches a MessagingException and logs a warning (should be an error IMO) f. example: 2021-06-01 15:25:08,374 WARN org.dspace.workflowbasic.BasicWorkflowServiceImpl @ *****@**:session_id=36E4D2106AB8E57280CFF8C6F198FCDC:notifyGroupofTask:cannot email user group_id=e16fefea-cef9-4ee5-8fbb-91006947b296 workflow_item_id=21105\colon; Invalid Addresses (com.sun.mail.smtp.SMTPAddressFailedException\colon; 550 5.1.1 <****@*********>... User unknown This means that if there is a single ePerson in the group with an email that is deleted then only those in the group that are iterated before the user with a deleted email will be notified. There is a simple fix for this, to have the try catch inside the for loop. I will provide a PR. https://github.com/DSpace/DSpace/blob/b574fb141a9424d3d0776647b0fcba01173fd837/dspace-api/src/main/java/org/dspace/workflowbasic/BasicWorkflowServiceImpl.java#L995-L1065 |
