Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 2.0.0

Request #13587 relay params like "NOTIFY=SUCCESS,FAILURE"
Submitted: 2008-04-04 11:31 UTC
From: alex2000 Assigned:
Status: Open Package: Mail (version 1.1.14)
PHP Version: 5.2.0 OS: SuSe 10.2
Roadmaps: (Not assigned)    
Subscription  


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 16 + 43 = ?

 
 [2008-04-04 11:31 UTC] alex2000 (Alexander Kant)
Description: ------------ Here: http://pear.php.net/package/Mail/docs/1.1.14/__filesource/fsource_Mail__Mail-1.1.14Mailsmtp.php.html , line 204 anybody have to place 4th parameter like $params and at the line 282 instead $this->_smtp->rcptTo($recipient) have to write $this->_smtp->rcptTo($recipient, $params) or somethins like that. Because I need this optional notify - flags and have to change it each time manually :( Can anybody do it for the next relase - or maybe another solution /way for the same problem. Thanks.

Comments

 [2008-04-06 07:19 UTC] doconnor (Daniel O'Connor)
Alexander; I'm not sure I understand your problem. You want to pass in extra params to: http://pear.php.net/package/Net_SMTP/docs/latest/Net_SMTP/Net_SMTP.html#methodrcptTo ? Or you want some kind of configuration option to allow a default setting for $params in the Mail_smtp driver? As an interim workaround, why not make a wrapper driver? class Mail_smtp_notify extends Mail_smtp { /** reimplemented send() method here which obeys your expected defaults */ }
 [2008-04-12 16:23 UTC] alex2000 (Alexander Kant)
I want simple to use a send() - method of Mail_smtp - class and have a chance to set the extra params, that can be give to a rcptTo() - method of Net_SMTP - class. To extend a Mail_smtp class and reimplemend a send() - method is a good and clean idea, but I hope that this possibility will be implemented in future (to get the extra param on a send() - method). Regards, Alex
 [2009-08-11 00:31 UTC] schmuwi (Uwe Mesecke)
The following patch has been added/updated: Patch Name: rcptTo-params-1.2.0 Revision: 1249932711 URL: http://pear.php.net/bugs/patch-display.php?bug=13587&patch=rcptTo-params-1.2.0&revision=1249932711&display=1
 [2009-08-11 00:37 UTC] schmuwi (Uwe Mesecke)
I added a patch that makes parameters for rcptTo() possible. This patch is based on the current beta 1.2.0b2 as there is now a method addServiceExtensionParameter() for setting parameters for the mailFrom() command. I changed addServiceExtensionParameter() so you can set the command to which you want to add the parameter: mail-from for mailFrom() (which is the default) or rcpt-to for rcptTo(). I have not tested the patch because I don't have a spare system at the moment but the change is fairly simple. Feedback is welcome... ;)
 [2010-10-06 21:18 UTC] sberthelot (Stephane Berthelot)
The attached patch mixes DOS and UNIX line endings but it make NOTIFY (DSN) work correctly (RFC1891) that has to be on RCPT TO line only. It seems this bug is very old, can it be fixed on next release cycle with current attached patch ? I tested it and it fixes the problem correctly