[Patches] [Patch #101151] expose public get_fqdn function in smtplib.py

noreply@sourceforge.net noreply@sourceforge.net
Fri, 11 Aug 2000 01:33:04 -0700


Patch #101151 has been updated. 

Project: 
Category: library
Status: Open
Summary: expose public get_fqdn function in smtplib.py

Follow-Ups:

Date: 2000-Aug-10 16:32
By: nowonder

Comment:
expose get_fqdn function rather than calling internal _get_fqdn_hostname function. Only call from helo() and ehlo() if needed. refer from documentation's mentioning of algorithm for fqdn to implementation of get_fqdn.

should there be docs for quotedata() and get_fqdn() in libsmtplib.tex?
-------------------------------------------------------

Date: 2000-Aug-11 00:58
By: twouters

Comment:
Looks okay, though perhaps the function should be named 'make_fqdn()' instead of 'get_fqdn' -- you're passing it a name to operate on, after all. That's just a personal preference.

I'll let it float for a few days to let someone else knowledgable in the area of SMTP give their opinion on the whole FQDN issue, but as far as I'm concerned, this patch is accepted. Am I the moral equivalent of Guido here ? I have no clue :-)


-------------------------------------------------------

Date: 2000-Aug-11 01:33
By: nowonder

Comment:
as I write in my mail to python-dev, I have found three other occurences of the algorithm. Maybe this should go in some more general place. Where?

BTW: I like make_fqdn() better.
-------------------------------------------------------

-------------------------------------------------------
For more info, visit:

http://sourceforge.net/patch/?func=detailpatch&patch_id=101151&group_id=5470