INTRODUCTION

----------------------
Bcc Logs all outgoing mail from your site.Send all outgoing mail to Bcc email id that passes through drupal_mail function

FEATURES

----------------
1. Bcc Logging all outgoing mail
2. Mail verification for bcc email id.
3. Activation/deactivation mail alert.
4. Editable mail template for verification, Activation and deactivation mail

INSTALLATION

--------------------
1. Copy bcc_mail_logger folder to modules directory.
2. At admin/build/modules enable the "Bcc Mail Logger" module.
4. Configure the module at admin/settings/bccmaillogger.

Issue with Phpmailer module

-----------------------------------
if you want to use this module with Phpmailer module then you need to apply following patch for it.
Phpmailer module Bug: The Cc and Bcc destination address fields are not the correct case .

CREDITS

-------------
This project has been sponsored by: Stigasoft

Comments

sarjeet.singh’s picture

sarjeet.singh commented

sandbox project link: http://drupal.org/sandbox/sarjeet.singh/1191318

sarjeet.singh’s picture

sarjeet.singh commented

I have submitted the project to sandbox and URL is: http://drupal.org/sandbox/sarjeet.singh/1191318
Please review it.

awolfey’s picture

awolfey commented
Status: Needs review » Needs work

I happened across this needing just this module.

I can confirm it works, and the code looks pretty good.

I would recommend:

  • Use "email address" instead of "email id".
  • Run through coder and coder tough love to fix commenting. The comments are missing some punctuation and capitalization.
  • I would just "return;" at the end of function _bcc_mail_logger_mail_text().
  • Exclude your verification emails from being copied. The current copy address gets the confirm link for the new address.
  • I don't think you need to check_plain(arg(1)) in bcc_mail_logger_acknowledgement().
  • If the validation link fails to validate, it could be because the link is wrong, not just expired.
  • Add a WARNING to the README.txt that using this module may cause access restricted information (including passwords and pass reset links and who knows what else) to go to the person who receives the logs regardless of that persons user role, permissions, etc. This is really a pretty big risk in my opinion. If user 1 requests a new password, the bcc recipient also gets the link.
  • You may want to add a feature to prevent users in certain roles from having their mails bcc'd.
awolfey’s picture

awolfey commented

Also, if the bcc header is already being used (there are other modules that use it), this will wipe our the existing bcc recipients.

sarjeet.singh’s picture

sarjeet.singh commented

Hi Aaron,
First of all thanks for your feedback on our module. Your comments are very valuable and does makes sense. I will surely update my module as per your comments.

Thanks a ton.

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Status: Needs work » Closed (won't fix)

No activity in several months. Reopen and set the status to "needs review" if you are still pursuing this application.