[issue19678] smtpd.py: channel should be passed to process_message

2018-01-09 Thread Barry A. Warsaw
Barry A. Warsaw added the comment: I'm going to close this as won't fix since smtpd.py is deprecated, and there's little chance that folks are still interested in working on it. See aiosmtpd as a much better third party replacement. -- resolution: -> wont fix

[issue19678] smtpd.py: channel should be passed to process_message

2017-12-30 Thread Sanyam Khurana
Change by Sanyam Khurana : -- keywords: +patch pull_requests: +4931 stage: needs patch -> patch review ___ Python tracker

[issue19678] smtpd.py: channel should be passed to process_message

2017-12-10 Thread Sanyam Khurana
Sanyam Khurana added the comment: Leslie, would you like to work on a patch for this? -- nosy: +CuriousLearner ___ Python tracker

[issue19678] smtpd.py: channel should be passed to process_message

2015-05-17 Thread R. David Murray
R. David Murray added the comment: At this point (3.5) we have settled on having process_message be required to have a **kwargs parameter in its signature in order to support new features (ex: enable_SMTPUTF8). So the solution here would be to use inspect.signature to find out if

[issue19678] smtpd.py: channel should be passed to process_message

2013-11-25 Thread R. David Murray
R. David Murray added the comment: I think this is reasonable. A patch would be welcome. You could use inspect. When I had to do something similar I just did the call inside a try/except, caught TypeError and retried without the extra argument. See the __init__ of

[issue19678] smtpd.py: channel should be passed to process_message

2013-11-21 Thread Leslie P. Polzer
New submission from Leslie P. Polzer: process_message needs to have access to the channel state since it needs to make decisions based on the authentication or transport associated with the channel. It should be either the first or the last arg. I can provide a patch for this. Should