Currently the module uses the default "from" address for the site.
It would be nice if the admin could set a different address.
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | 545044-10-allow-email-from-address.patch | 4.35 KB | chris burge |
| #8 | 545044-8-allow-email-from-address.patch | 3.5 KB | chris burge |
Comments
Comment #1
gregglesIn retrospect, this seems like code bloat. I'm not sure this feature belongs in comment notify.
Comment #2
gregglesI guess nobody's arguing for it/against it except for me. Won't fix it is!
Comment #3
WildBill commentedI'm in favor of this feature. In particular, it would be cool if the notification was from "MyName" rather than from "admin@mydomain.com".
Comment #4
doublejosh commentedDefinitely a +1 for me!
We want to use a site default email that's more valid and has a person watching it. But the comment notify emails need to come from no-reply@ so people know not to reply and we don't get overwhelmed by bounces from bad comment emails!
Seems like a no brainer.
Comment #5
Shai commented+ 1 for a no-reply email address.
Comment #6
gregglesSure - anyone want to post a patch?
Comment #7
ADrupalUser commented+1.
I allow anonymous users to subscribe with comment notify, but if they put in a fake email address then the default "From" account constantly gets bounce backs. Very annoying.
Comment #8
chris burge commentedPatch attached that adds requested functionality.
Comment #9
chris burge commentedI should add that if you're concerned about exposing your admin e-mail address, then you should look at the mail header of an outgoing message to make sure it isn't be exposed as the 'return-path' address or elsewhere.
Comment #10
chris burge commentedUpdated patch. The previous patch didn't allow for the site admin e-mail address to be used, which is the existing behavior. This updated patch adds this as an option, which may be desirable in certain circumstances.