Another thing I found is:
in SendEmail.pm. This shows the techs email address to the client. I’ve
removed this, any reason for it being included?
It’s often useful information, especially for internal uses. It’s perfectly
OK to use it if your site doesn’t want it there.
Might be a good config.pm option, as quite often you don’t want end
users knowning a real email address, but rather they should always use a
support one (make sure everything goes through the system).
I could see it for comments between techs, or debug mode though.