I ran into this bug the other day as a patron! :-)
Email::Simple is definitely mangling the headers of emails sent from the OPAC (similar to bug 1801163, which Jason S, Galen, and Jason B fixed).
Here is a branch which signs off on Mike's fix, moves the OPAC mailing to Email::MIME, and adds some tests: user/sandbergja/lp1955079_email_records_fix
Adding pullrequest to get more eyes on this, but please note that I have only tested in my local dev setup; this still should be tested on a real server that sends real emails before getting merged.
I ran into this bug the other day as a patron! :-)
Email::Simple is definitely mangling the headers of emails sent from the OPAC (similar to bug 1801163, which Jason S, Galen, and Jason B fixed).
Here is a branch which signs off on Mike's fix, moves the OPAC mailing to Email::MIME, and adds some tests: user/sandbergja /lp1955079_ email_records_ fix
Here's a link: https:/ /git.evergreen- ils.org/ ?p=working/ Evergreen. git;a=shortlog; h=refs/ heads/user/ sandbergja/ lp1955079_ email_records_ fix
Adding pullrequest to get more eyes on this, but please note that I have only tested in my local dev setup; this still should be tested on a real server that sends real emails before getting merged.