CUNY Academic Commons

It appears that the MailPoet wrapper for PHPMailer sets the Sender variable (which determines Return-Path) using the 'bounce_email' config setting. We might consider filtering the 'wysija' option as it comes out of the databsae, and forcing 'commons-no-reply@gc.cuny.edu' for the 'bounce_email' setting across the whole network. This seems like a somewhat lighter solution than forcing the use of wp_mail().

Decided to go with your method of setting the 'bounce_email' option. MailPoet has a funky DB options API, so I decided to stick with their API to set the 'Return-Path' header.

Fixed in commit 239cc03.

Just got a report of a bounced email with no Return-Path set. Here are the headers (I've removed the recipient email address for privacy):

Return-Path:
Received: from pps.filterd (proofsender1.gc.cuny.edu [127.0.0.1])
    by proofsender1.gc.cuny.edu (8.15.0.59/8.15.0.59) with SMTP id u861amMl026996
    for <...>; Mon, 5 Sep 2016 21:40:21 -0400
Received: from outmail.gc.cuny.edu (outmail.gc.cuny.edu [146.96.128.57])
    by proofsender1.gc.cuny.edu with ESMTP id 257qw4jkqn-27
    for <...>; Mon, 05 Sep 2016 21:40:20 -0400
Received: from lw1c.gc.cuny.edu (172.29.28.143) by WA10B.gc.cuny.edu
 (172.29.28.90) with Microsoft SMTP Server id 8.3.465.0; Mon, 5 Sep 2016
 21:40:17 -0400
Received: by lw1c.gc.cuny.edu (Postfix, from userid 48)    id 2CE3E80D0648; Mon,
  5 Sep 2016 21:40:17 -0400 (EDT)
To: <...>
Subject: Monday 9/12: MARX AND CAPITAL: THE CONCEPT, THE BOOK, THE HISTORY A SERIES OF MONDAY EVENING LECTURES IN POLITICAL ECONOMY BY DAVID HARVEY
X-PHP-Originating-Script: 10009264:class.phpmailer.php
Date: Tue, 6 Sep 2016 01:40:17 +0000
From: "The Center for Place, Culture and Politics" <pcp@gc.cuny.edu>
Reply-To: "The Center for Place, Culture and Politics" <pcp@gc.cuny.edu>
Message-ID: <ODkzMzUzOQWY1281SI337JA1109MSMTQ3MzEyNjAxNzcyNTU1@pcp.gc.cuny.edu>
X-Priority: 3
X-Mailer: PHPMailer (phpmailer.sourceforge.net) [version 2.0.4]
X-email_id: 337
MIME-Version: 1.0
Content-Type: multipart/alternative;
    boundary="b1_883d9808923439ac41567bbacd608f0b" 
Return-Path: apache@lw1c.gc.cuny.edu
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2016-09-05_11:,,
 signatures=0
X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 spamscore=0 suspectscore=87
 malwarescore=0 phishscore=0 adultscore=0 bulkscore=100 classifier=spam
 adjust=0 reason=mlx scancount=1 engine=8.0.1-1604210000
 definitions=main-1609060022

This is generated by MailPoet, right? Maybe your whitespace trimming in https://github.com/cuny-academic-commons/cac/commit/239cc0349e89295ad8a828b3e1531cba9104d474 is failng?

It seems that we also have to set MailPoet's ms_bounce_email option for our custom Return-Path header to take effect.

Should be fixed this time in commit 52d84bb.