Currently the module uses the default "from" address for the site.

It would be nice if the admin could set a different address.

Comments

greggles’s picture

In retrospect, this seems like code bloat. I'm not sure this feature belongs in comment notify.

greggles’s picture

Status: Active » Closed (won't fix)

I guess nobody's arguing for it/against it except for me. Won't fix it is!

WildBill’s picture

I'm in favor of this feature. In particular, it would be cool if the notification was from "MyName" rather than from "admin@mydomain.com".

doublejosh’s picture

Definitely 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.

Shai’s picture

+ 1 for a no-reply email address.

greggles’s picture

Status: Closed (won't fix) » Active

Sure - anyone want to post a patch?

ADrupalUser’s picture

Issue summary: View changes

+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.

chris burge’s picture

Status: Active » Needs review
StatusFileSize
new3.5 KB

Patch attached that adds requested functionality.

chris burge’s picture

I 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.

chris burge’s picture

StatusFileSize
new4.35 KB

Updated 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.