homepage

This issue tracker has been migrated to GitHub , and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: mailbox: It would be nice to move mailbox.Message from legacy email.message.Message API to new EmailMessage API
Type: enhancement Stage:
Components: email Versions: Python 3.8
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: Nosy List: barry, maxking, r.david.murray
Priority: normal Keywords:

Created on 2018年03月01日 05:44 by maxking, last changed 2022年04月11日 14:58 by admin.

Messages (4)
msg313082 - (view) Author: Abhilash Raj (maxking) * (Python committer) Date: 2018年03月01日 05:44
Since Python 3.6 the new EmailMessage API seems to be the default but mailbox.Message still subclasses from the old email.message.Message API.
It would be nice to get EmailMessage from mailbox so that one can rely on the new methods and content managers. Also, while it is possible to pass a constructor to mailbox.mbox to get the new EmailMessage style message, it is different from mailbox.Message which has some extra methods.
msg313083 - (view) Author: Abhilash Raj (maxking) * (Python committer) Date: 2018年03月01日 06:00
I should also mention probably that I went ahead and made changes to make this work and I can pass most of the tests, one is failing due to missing `\n` character in the header's value obtained from `msg.get_all('Received')`.
I can make a pull request on Github if that would be a good place to start the discussion instead.
msg313120 - (view) Author: R. David Murray (r.david.murray) * (Python committer) Date: 2018年03月02日 01:25
This is a reasonable request (a good way to use EmailMessage), but EmailMessage is not yet the default even for the email package. You have to specify policy=default to get it (or use it to construct a message).
Mailbox probably needs some backward compatible way for the user to specify the policy.
msg313122 - (view) Author: Abhilash Raj (maxking) * (Python committer) Date: 2018年03月02日 02:29
Can you suggest what would be the best way go about doing that?
Given that mailbox.Message subclasses from Message, I am not sure how to proceed about this to make this configurable with a flag.
I haven't yet made a PR, but this[1] is the URL to the change I made.
[1]: https://github.com/python/cpython/compare/master...maxking:fix-mailbox 
History
Date User Action Args
2022年04月11日 14:58:58adminsetgithub: 77156
2018年03月02日 02:29:36maxkingsetmessages: + msg313122
2018年03月02日 01:25:41r.david.murraysetmessages: + msg313120
versions: + Python 3.8, - Python 3.6
2018年03月01日 06:00:31maxkingsetmessages: + msg313083
2018年03月01日 05:44:18maxkingcreate

AltStyle によって変換されたページ (->オリジナル) /