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

Request #16066 need to retrieve the greeting sent by the server
Submitted: 2009-04-03 22:04 UTC
From: kguest Assigned: jon
Status: Closed Package: Net_SMTP (version 1.3.2)
PHP Version: 5.2.5 OS:
Roadmaps: (Not assigned)    
Subscription  


 [2009-04-03 22:04 UTC] kguest (Ken Guest)
Description: ------------ need to retrieve the greeting sent by the server - from this you can determine which actual mail server you have interacted with; very useful for tracking if you're connecting through a round-robin of mail servers! similar patch coming for the mail package.

Comments

 [2009-04-03 22:05 UTC] kguest (Ken Guest)
The following patch has been added/updated: Patch Name: Net_SMTP-SMTP.php.greeting.diff Revision: 1238778319 URL: http://pear.php.net/bugs/patch-display.php?bug=$bug&patch=$patch_name&revision=$e&display=1
 [2009-04-03 22:08 UTC] kguest (Ken Guest)
the patch given here is a dependency of that given for http://pear.php.net/bugs/bug.php?id=16067
 [2009-04-04 08:34 UTC] jon (Jon Parise)
-Status: Open +Status: Assigned -Assigned To: +Assigned To: jon
 [2009-04-07 10:47 UTC] jon (Jon Parise)
-Status: Assigned +Status: Feedback
It is already possible to retrieve the SMTP server's response arguments via the getResponse() method. Is that not sufficient for your use case? Also, with your given patch, the greeting string is set to the entire (first) response line. Would it be better to set it to just the argument portion (without the code)?
 [2009-04-07 22:43 UTC] kguest (Ken Guest)
The following patch has been added/updated: Patch Name: Net_SMTP-SMTP.php.greeting.v2.diff Revision: 1239126202 URL: http://pear.php.net/bugs/patch-display.php?bug=$bug&patch=$patch_name&revision=$e&display=1
 [2009-04-07 22:50 UTC] kguest (Ken Guest)
now that I'm a bit more clued in, here's a better way of doing this ;-) (as hinted at in your comment for request 16063) this uses getResponse in the connect method - much cleaner and with mimimum performance hit!
 [2009-05-25 02:55 UTC] jon (Jon Parise)
-Status: Feedback +Status: Assigned
That looks pretty reasonable. I'll try to incorporate this into the Net_SMTP package soon.
 [2009-05-26 05:32 UTC] jon (Jon Parise)
-Status: Assigned +Status: Closed
This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.