[issue3802] smtpd.py __getaddr insufficient handling

[Barry A. Warsaw]

Barry A. Warsaw  added the comment:

I'm closing this as won't fix since smtpd.py is deprecated and will likely not 
get any future development.  Please see aiosmtpd as a much better third party 
replacement.

--
nosy: +barry
resolution:  -> wont fix
stage: test needed -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3802] smtpd.py __getaddr insufficient handling

[Mark Lawrence]

Changes by Mark Lawrence :


--
nosy:  -BreamoreBoy

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3802] smtpd.py __getaddr insufficient handling

[Leslie P. Polzer]

Changes by Leslie P. Polzer :


--
nosy: +lpolzer

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3802] smtpd.py __getaddr insufficient handling

[Petri Lehtinen]

Petri Lehtinen  added the comment:

AFAIK, the extra MAIL FROM and RCPT TO parameters are only valid for ESMTP. 
smtpd doesn't currently handle ESMTP, so this should not be a problem.

--
nosy: +petri.lehtinen
versions: +Python 2.7, Python 3.2, Python 3.3 -Python 2.6

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3802] smtpd.py __getaddr insufficient handling

[Mark Lawrence]

Mark Lawrence  added the comment:

Note there is a patch inline, not sure if a doc patch is needed for this.

--
keywords: +patch
nosy: +BreamoreBoy
stage:  -> unit test needed
type:  -> behavior

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3802] smtpd.py __getaddr insufficient handling

[Marcus CM]

New submission from Marcus CM <[EMAIL PROTECTED]>:

The __getaddr does not handle certain valid MAIL FROM well :

For eg,

<[EMAIL PROTECTED]> SIZE= AUTH=<>

would result in a mismatch of bracket handling.


Suggested fix is :-

def __getaddr(self, keyword, arg):
address = None
keylen = len(keyword)
if arg[:keylen].upper() == keyword:
address = arg[keylen:].strip()
if not address:
pass

# Marcus fix :
i  = address.count("<")
ii = address.count(">")
if i != ii :
address = None
return address

# Marcus remark : bug if : <[EMAIL PROTECTED]> SIZE=6092 AUTH=<>
elif address[0] == '<' and address[-1] == '>' and address !
= '<>':
# Addresses can be in the form <[EMAIL PROTECTED]> but 
watch out
# for null address, e.g. <>

if address.count("<") == 1 :
address = address[1:-1]

return address

--
messages: 72763
nosy: [EMAIL PROTECTED]
severity: normal
status: open
title: smtpd.py __getaddr insufficient handling
versions: Python 2.6

___
Python tracker <[EMAIL PROTECTED]>

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com