It seems like good practice to add in X-Originating-IP header in the email identifying the original sender. This will allow the recipient of a mail sent by this module to track down spammers without help from the site admin.

See http://en.wikipedia.org/wiki/X-Originating-IP

CommentFileSizeAuthor
#3 x-originating-ip-2.patch1.37 KBc960657
#1 x-originating-ip-1-D6.patch416 bytesc960657
x-originating-ip-1.patch452 bytesc960657

Comments

c960657’s picture

c960657 commented
Version: 7.x-1.0-rc4 » 7.x-1.x-dev
StatusFileSize
new x-originating-ip-1-D6.patch416 bytes

D6 backport.

john.oltman’s picture

john.oltman commented
Status: Needs review » Needs work

This should be optional via a check box on the configuration form. If you can provide patches to 7.x-1.0 and 6.x-1.x-dev that provide this as an option instead of making it mandatory, I would be inclined to support it. Thanks!

c960657’s picture

c960657 commented
Status: Needs work » Needs review
StatusFileSize
new x-originating-ip-2.patch1.37 KB

New patch with a checkbox. I think TRUE is a reasonable default - the user is requested to enter his own email address, so he is not under the impression that he is anonymous.

deekayen’s picture

deekayen commented
Status: Needs review » Closed (duplicate)

I just threw this together as a quick utility:

https://drupal.org/project/x_originating_ip