Empty envelope sender in daemon notifications - Githubissues

znuny / Znuny

Znuny/Znuny LTS is a fork of the ((OTRS)) Community Edition, one of the most flexible web-based ticketing systems used for Customer Service, Help Desk, IT Service Management.

https://www.znuny.org

GNU General Public License v3.0

343 stars 83 forks source link

Empty envelope sender in daemon notifications #409

Open pboguslawski opened 1 year ago

pboguslawski commented 1 year ago

Proposed change

Znuny daemon notifications are sent with nonempty envelope sender; in case of delivery failures bounces may be unnecessary generated.

No such problem in regular notifications

https://github.com/znuny/Znuny/blob/dev/Kernel/System/Ticket/Event/NotificationEvent/Transport/Email.pm#L186

This mod sets empty envelope sender for daemon notifications also.

Type of change

'1 - 🐞 bug 🐞' - Bugfix (non-breaking change which fixes an issue)

Additional information

Replaces: https://github.com/znuny/Znuny/pull/161 Related: https://datatracker.ietf.org/doc/html/rfc5321#section-4.1.1.2 Author-Change-Id: IB#1111343

Checklist

[x] The code change is tested and works locally.(❗)
[x] There is no commented out code in this PR.(❕)
[ ] You improved or added new unit tests.(❕)
[x] Local ZnunyCodePolicy passed.(❕)
[x] Local UnitTests / Selenium passed.(❕)
[ ] GitHub workflow CI (UnitTests / Selenium) passed.(❗)