From a0c0a4a2616e83084052a34c9806b3308f5799db Mon Sep 17 00:00:00 2001 From: Benjamin Peterson Date: Thu, 12 Jun 2008 22:15:50 +0000 Subject: remove the rfc822 module --- Doc/library/netdata.rst | 1 - Doc/library/rfc822.rst | 351 ---------------- Lib/rfc822.py | 1003 ---------------------------------------------- Lib/test/test_pyclbr.py | 1 - Lib/test/test_rfc822.py | 256 ------------ Lib/test/test_urllib2.py | 4 +- Misc/NEWS | 2 + 7 files changed, 4 insertions(+), 1614 deletions(-) delete mode 100644 Doc/library/rfc822.rst delete mode 100644 Lib/rfc822.py delete mode 100644 Lib/test/test_rfc822.py diff --git a/Doc/library/netdata.rst b/Doc/library/netdata.rst index ce35bfa..4915016 100644 --- a/Doc/library/netdata.rst +++ b/Doc/library/netdata.rst @@ -16,7 +16,6 @@ on the Internet. mailcap.rst mailbox.rst mimetypes.rst - rfc822.rst base64.rst binhex.rst binascii.rst diff --git a/Doc/library/rfc822.rst b/Doc/library/rfc822.rst deleted file mode 100644 index 2b0ca30..0000000 --- a/Doc/library/rfc822.rst +++ /dev/null @@ -1,351 +0,0 @@ - -:mod:`rfc822` --- Parse RFC 2822 mail headers -============================================= - -.. module:: rfc822 - :synopsis: Parse 2822 style mail messages. - :deprecated: - - -.. deprecated:: 2.3 - The :mod:`email` package should be used in preference to the :mod:`rfc822` - module. This module is present only to maintain backward compatibility. - -This module defines a class, :class:`Message`, which represents an "email -message" as defined by the Internet standard :rfc:`2822`. [#]_ Such messages -consist of a collection of message headers, and a message body. This module -also defines a helper class :class:`AddressList` for parsing :rfc:`2822` -addresses. Please refer to the RFC for information on the specific syntax of -:rfc:`2822` messages. - -.. index:: module: mailbox - -The :mod:`mailbox` module provides classes to read mailboxes produced by -various end-user mail programs. - - -.. class:: Message(file[, seekable]) - - A :class:`Message` instance is instantiated with an input object as parameter. - Message relies only on the input object having a :meth:`readline` method; in - particular, ordinary file objects qualify. Instantiation reads headers from the - input object up to a delimiter line (normally a blank line) and stores them in - the instance. The message body, following the headers, is not consumed. - - This class can work with any input object that supports a :meth:`readline` - method. If the input object has seek and tell capability, the - :meth:`rewindbody` method will work; also, illegal lines will be pushed back - onto the input stream. If the input object lacks seek but has an :meth:`unread` - method that can push back a line of input, :class:`Message` will use that to - push back illegal lines. Thus this class can be used to parse messages coming - from a buffered stream. - - The optional *seekable* argument is provided as a workaround for certain stdio - libraries in which :cfunc:`tell` discards buffered data before discovering that - the :cfunc:`lseek` system call doesn't work. For maximum portability, you - should set the seekable argument to zero to prevent that initial :meth:`tell` - when passing in an unseekable object such as a file object created from a socket - object. - - Input lines as read from the file may either be terminated by CR-LF or by a - single linefeed; a terminating CR-LF is replaced by a single linefeed before the - line is stored. - - All header matching is done independent of upper or lower case; e.g. - ``m['From']``, ``m['from']`` and ``m['FROM']`` all yield the same result. - - -.. class:: AddressList(field) - - You may instantiate the :class:`AddressList` helper class using a single string - parameter, a comma-separated list of :rfc:`2822` addresses to be parsed. (The - parameter ``None`` yields an empty list.) - - -.. function:: quote(str) - - Return a new string with backslashes in *str* replaced by two backslashes and - double quotes replaced by backslash-double quote. - - -.. function:: unquote(str) - - Return a new string which is an *unquoted* version of *str*. If *str* ends and - begins with double quotes, they are stripped off. Likewise if *str* ends and - begins with angle brackets, they are stripped off. - - -.. function:: parseaddr(address) - - Parse *address*, which should be the value of some address-containing field such - as :mailheader:`To` or :mailheader:`Cc`, into its constituent "realname" and - "email address" parts. Returns a tuple of that information, unless the parse - fails, in which case a 2-tuple ``(None, None)`` is returned. - - -.. function:: dump_address_pair(pair) - - The inverse of :meth:`parseaddr`, this takes a 2-tuple of the form ``(realname, - email_address)`` and returns the string value suitable for a :mailheader:`To` or - :mailheader:`Cc` header. If the first element of *pair* is false, then the - second element is returned unmodified. - - -.. function:: parsedate(date) - - Attempts to parse a date according to the rules in :rfc:`2822`. however, some - mailers don't follow that format as specified, so :func:`parsedate` tries to - guess correctly in such cases. *date* is a string containing an :rfc:`2822` - date, such as ``'Mon, 20 Nov 1995 19:12:08 -0500'``. If it succeeds in parsing - the date, :func:`parsedate` returns a 9-tuple that can be passed directly to - :func:`time.mktime`; otherwise ``None`` will be returned. Note that indexes 6, - 7, and 8 of the result tuple are not usable. - - -.. function:: parsedate_tz(date) - - Performs the same function as :func:`parsedate`, but returns either ``None`` or - a 10-tuple; the first 9 elements make up a tuple that can be passed directly to - :func:`time.mktime`, and the tenth is the offset of the date's timezone from UTC - (which is the official term for Greenwich Mean Time). (Note that the sign of - the timezone offset is the opposite of the sign of the ``time.timezone`` - variable for the same timezone; the latter variable follows the POSIX standard - while this module follows :rfc:`2822`.) If the input string has no timezone, - the last element of the tuple returned is ``None``. Note that indexes 6, 7, and - 8 of the result tuple are not usable. - - -.. function:: mktime_tz(tuple) - - Turn a 10-tuple as returned by :func:`parsedate_tz` into a UTC timestamp. If - the timezone item in the tuple is ``None``, assume local time. Minor - deficiency: this first interprets the first 8 elements as a local time and then - compensates for the timezone difference; this may yield a slight error around - daylight savings time switch dates. Not enough to worry about for common use. - - -.. seealso:: - - Module :mod:`email` - Comprehensive email handling package; supersedes the :mod:`rfc822` module. - - Module :mod:`mailbox` - Classes to read various mailbox formats produced by end-user mail programs. - - -.. _message-objects: - -Message Objects ---------------- - -A :class:`Message` instance has the following methods: - - -.. method:: Message.rewindbody() - - Seek to the start of the message body. This only works if the file object is - seekable. - - -.. method:: Message.isheader(line) - - Returns a line's canonicalized fieldname (the dictionary key that will be used - to index it) if the line is a legal :rfc:`2822` header; otherwise returns - ``None`` (implying that parsing should stop here and the line be pushed back on - the input stream). It is sometimes useful to override this method in a - subclass. - - -.. method:: Message.islast(line) - - Return true if the given line is a delimiter on which Message should stop. The - delimiter line is consumed, and the file object's read location positioned - immediately after it. By default this method just checks that the line is - blank, but you can override it in a subclass. - - -.. method:: Message.iscomment(line) - - Return ``True`` if the given line should be ignored entirely, just skipped. By - default this is a stub that always returns ``False``, but you can override it in - a subclass. - - -.. method:: Message.getallmatchingheaders(name) - - Return a list of lines consisting of all headers matching *name*, if any. Each - physical line, whether it is a continuation line or not, is a separate list - item. Return the empty list if no header matches *name*. - - -.. method:: Message.getfirstmatchingheader(name) - - Return a list of lines comprising the first header matching *name*, and its - continuation line(s), if any. Return ``None`` if there is no header matching - *name*. - - -.. method:: Message.getrawheader(name) - - Return a single string consisting of the text after the colon in the first - header matching *name*. This includes leading whitespace, the trailing - linefeed, and internal linefeeds and whitespace if there any continuation - line(s) were present. Return ``None`` if there is no header matching *name*. - - -.. method:: Message.getheader(name[, default]) - - Return a single string consisting of the last header matching *name*, - but strip leading and trailing whitespace. - Internal whitespace is not stripped. The optional *default* argument can be - used to specify a different default to be returned when there is no header - matching *name*; it defaults to ``None``. - This is the preferred way to get parsed headers. - - -.. method:: Message.get(name[, default]) - - An alias for :meth:`getheader`, to make the interface more compatible with - regular dictionaries. - - -.. method:: Message.getaddr(name) - - Return a pair ``(full name, email address)`` parsed from the string returned by - ``getheader(name)``. If no header matching *name* exists, return ``(None, - None)``; otherwise both the full name and the address are (possibly empty) - strings. - - Example: If *m*'s first :mailheader:`From` header contains the string - ``'jack@cwi.nl (Jack Jansen)'``, then ``m.getaddr('From')`` will yield the pair - ``('Jack Jansen', 'jack@cwi.nl')``. If the header contained ``'Jack Jansen - '`` instead, it would yield the exact same result. - - -.. method:: Message.getaddrlist(name) - - This is similar to ``getaddr(list)``, but parses a header containing a list of - email addresses (e.g. a :mailheader:`To` header) and returns a list of ``(full - name, email address)`` pairs (even if there was only one address in the header). - If there is no header matching *name*, return an empty list. - - If multiple headers exist that match the named header (e.g. if there are several - :mailheader:`Cc` headers), all are parsed for addresses. Any continuation lines - the named headers contain are also parsed. - - -.. method:: Message.getdate(name) - - Retrieve a header using :meth:`getheader` and parse it into a 9-tuple compatible - with :func:`time.mktime`; note that fields 6, 7, and 8 are not usable. If - there is no header matching *name*, or it is unparsable, return ``None``. - - Date parsing appears to be a black art, and not all mailers adhere to the - standard. While it has been tested and found correct on a large collection of - email from many sources, it is still possible that this function may - occasionally yield an incorrect result. - - -.. method:: Message.getdate_tz(name) - - Retrieve a header using :meth:`getheader` and parse it into a 10-tuple; the - first 9 elements will make a tuple compatible with :func:`time.mktime`, and the - 10th is a number giving the offset of the date's timezone from UTC. Note that - fields 6, 7, and 8 are not usable. Similarly to :meth:`getdate`, if there is - no header matching *name*, or it is unparsable, return ``None``. - -:class:`Message` instances also support a limited mapping interface. In -particular: ``m[name]`` is like ``m.getheader(name)`` but raises :exc:`KeyError` -if there is no matching header; and ``len(m)``, ``m.get(name[, default])``, -``m.__contains__(name)``, ``m.keys()``, ``m.values()`` ``m.items()``, and -``m.setdefault(name[, default])`` act as expected, with the one difference -that :meth:`setdefault` uses an empty string as the default value. -:class:`Message` instances also support the mapping writable interface ``m[name] -= value`` and ``del m[name]``. :class:`Message` objects do not support the -:meth:`clear`, :meth:`copy`, :meth:`popitem`, or :meth:`update` methods of the -mapping interface. (Support for :meth:`get` and :meth:`setdefault` was only -added in Python 2.2.) - -Finally, :class:`Message` instances have some public instance variables: - - -.. attribute:: Message.headers - - A list containing the entire set of header lines, in the order in which they - were read (except that setitem calls may disturb this order). Each line contains - a trailing newline. The blank line terminating the headers is not contained in - the list. - - -.. attribute:: Message.fp - - The file or file-like object passed at instantiation time. This can be used to - read the message content. - - -.. attribute:: Message.unixfrom - - The Unix ``From`` line, if the message had one, or an empty string. This is - needed to regenerate the message in some contexts, such as an ``mbox``\ -style - mailbox file. - - -.. _addresslist-objects: - -AddressList Objects -------------------- - -An :class:`AddressList` instance has the following methods: - - -.. method:: AddressList.__len__() - - Return the number of addresses in the address list. - - -.. method:: AddressList.__str__() - - Return a canonicalized string representation of the address list. Addresses are - rendered in "name" form, comma-separated. - - -.. method:: AddressList.__add__(alist) - - Return a new :class:`AddressList` instance that contains all addresses in both - :class:`AddressList` operands, with duplicates removed (set union). - - -.. method:: AddressList.__iadd__(alist) - - In-place version of :meth:`__add__`; turns this :class:`AddressList` instance - into the union of itself and the right-hand instance, *alist*. - - -.. method:: AddressList.__sub__(alist) - - Return a new :class:`AddressList` instance that contains every address in the - left-hand :class:`AddressList` operand that is not present in the right-hand - address operand (set difference). - - -.. method:: AddressList.__isub__(alist) - - In-place version of :meth:`__sub__`, removing addresses in this list which are - also in *alist*. - -Finally, :class:`AddressList` instances have one public instance variable: - - -.. attribute:: AddressList.addresslist - - A list of tuple string pairs, one per address. In each member, the first is the - canonicalized name part, the second is the actual route-address (``'@'``\ - -separated username-host.domain pair). - -.. rubric:: Footnotes - -.. [#] This module originally conformed to :rfc:`822`, hence the name. Since then, - :rfc:`2822` has been released as an update to :rfc:`822`. This module should be - considered :rfc:`2822`\ -conformant, especially in cases where the syntax or - semantics have changed since :rfc:`822`. - diff --git a/Lib/rfc822.py b/Lib/rfc822.py deleted file mode 100644 index 1ad2d01..0000000 --- a/Lib/rfc822.py +++ /dev/null @@ -1,1003 +0,0 @@ -"""RFC 2822 message manipulation. - -Note: This is only a very rough sketch of a full RFC-822 parser; in particular -the tokenizing of addresses does not adhere to all the quoting rules. - -Note: RFC 2822 is a long awaited update to RFC 822. This module should -conform to RFC 2822, and is thus mis-named (it's not worth renaming it). Some -effort at RFC 2822 updates have been made, but a thorough audit has not been -performed. Consider any RFC 2822 non-conformance to be a bug. - - RFC 2822: http://www.faqs.org/rfcs/rfc2822.html - RFC 822 : http://www.faqs.org/rfcs/rfc822.html (obsolete) - -Directions for use: - -To create a Message object: first open a file, e.g.: - - fp = open(file, 'r') - -You can use any other legal way of getting an open file object, e.g. use -sys.stdin or call os.popen(). Then pass the open file object to the Message() -constructor: - - m = Message(fp) - -This class can work with any input object that supports a readline method. If -the input object has seek and tell capability, the rewindbody method will -work; also illegal lines will be pushed back onto the input stream. If the -input object lacks seek but has an `unread' method that can push back a line -of input, Message will use that to push back illegal lines. Thus this class -can be used to parse messages coming from a buffered stream. - -The optional `seekable' argument is provided as a workaround for certain stdio -libraries in which tell() discards buffered data before discovering that the -lseek() system call doesn't work. For maximum portability, you should set the -seekable argument to zero to prevent that initial \code{tell} when passing in -an unseekable object such as a a file object created from a socket object. If -it is 1 on entry -- which it is by default -- the tell() method of the open -file object is called once; if this raises an exception, seekable is reset to -0. For other nonzero values of seekable, this test is not made. - -To get the text of a particular header there are several methods: - - str = m.getheader(name) - str = m.getrawheader(name) - -where name is the name of the header, e.g. 'Subject'. The difference is that -getheader() strips the leading and trailing whitespace, while getrawheader() -doesn't. Both functions retain embedded whitespace (including newlines) -exactly as they are specified in the header, and leave the case of the text -unchanged. - -For addresses and address lists there are functions - - realname, mailaddress = m.getaddr(name) - list = m.getaddrlist(name) - -where the latter returns a list of (realname, mailaddr) tuples. - -There is also a method - - time = m.getdate(name) - -which parses a Date-like field and returns a time-compatible tuple, -i.e. a tuple such as returned by time.localtime() or accepted by -time.mktime(). - -See the class definition for lower level access methods. - -There are also some utility functions here. -""" -# Cleanup and extensions by Eric S. Raymond - -import time - -__all__ = ["Message","AddressList","parsedate","parsedate_tz","mktime_tz"] - -_blanklines = ('\r\n', '\n') # Optimization for islast() - - -class Message: - """Represents a single RFC 2822-compliant message.""" - - def __init__(self, fp, seekable = 1): - """Initialize the class instance and read the headers.""" - if seekable == 1: - # Exercise tell() to make sure it works - # (and then assume seek() works, too) - try: - fp.tell() - except (AttributeError, IOError): - seekable = 0 - self.fp = fp - self.seekable = seekable - self.startofheaders = None - self.startofbody = None - # - if self.seekable: - try: - self.startofheaders = self.fp.tell() - except IOError: - self.seekable = 0 - # - self.readheaders() - # - if self.seekable: - try: - self.startofbody = self.fp.tell() - except IOError: - self.seekable = 0 - - def rewindbody(self): - """Rewind the file to the start of the body (if seekable).""" - if not self.seekable: - raise IOError("unseekable file") - self.fp.seek(self.startofbody) - - def readheaders(self): - """Read header lines. - - Read header lines up to the entirely blank line that terminates them. - The (normally blank) line that ends the headers is skipped, but not - included in the returned list. If a non-header line ends the headers, - (which is an error), an attempt is made to backspace over it; it is - never included in the returned list. - - The variable self.status is set to the empty string if all went well, - otherwise it is an error message. The variable self.headers is a - completely uninterpreted list of lines contained in the header (so - printing them will reproduce the header exactly as it appears in the - file). - """ - self.dict = {} - self.unixfrom = '' - self.headers = lst = [] - self.status = '' - headerseen = "" - firstline = 1 - startofline = unread = tell = None - if hasattr(self.fp, 'unread'): - unread = self.fp.unread - elif self.seekable: - tell = self.fp.tell - while 1: - if tell: - try: - startofline = tell() - except IOError: - startofline = tell = None - self.seekable = 0 - line = self.fp.readline() - if not line: - self.status = 'EOF in headers' - break - # Skip unix From name time lines - if firstline and line.startswith('From '): - self.unixfrom = self.unixfrom + line - continue - firstline = 0 - if headerseen and line[0] in ' \t': - # It's a continuation line. - lst.append(line) - x = (self.dict[headerseen] + "\n " + line.strip()) - self.dict[headerseen] = x.strip() - continue - elif self.iscomment(line): - # It's a comment. Ignore it. - continue - elif self.islast(line): - # Note! No pushback here! The delimiter line gets eaten. - break - headerseen = self.isheader(line) - if headerseen: - # It's a legal header line, save it. - lst.append(line) - self.dict[headerseen] = line[len(headerseen)+1:].strip() - continue - else: - # It's not a header line; throw it back and stop here. - if not self.dict: - self.status = 'No headers' - else: - self.status = 'Non-header line where header expected' - # Try to undo the read. - if unread: - unread(line) - elif tell: - self.fp.seek(startofline) - else: - self.status = self.status + '; bad seek' - break - - def isheader(self, line): - """Determine whether a given line is a legal header. - - This method should return the header name, suitably canonicalized. - You may override this method in order to use Message parsing on tagged - data in RFC 2822-like formats with special header formats. - """ - i = line.find(':') - if i > 0: - return line[:i].lower() - return None - - def islast(self, line): - """Determine whether a line is a legal end of RFC 2822 headers. - - You may override this method if your application wants to bend the - rules, e.g. to strip trailing whitespace, or to recognize MH template - separators ('--------'). For convenience (e.g. for code reading from - sockets) a line consisting of \r\n also matches. - """ - return line in _blanklines - - def iscomment(self, line): - """Determine whether a line should be skipped entirely. - - You may override this method in order to use Message parsing on tagged - data in RFC 2822-like formats that support embedded comments or - free-text data. - """ - return False - - def getallmatchingheaders(self, name): - """Find all header lines matching a given header name. - - Look through the list of headers and find all lines matching a given - header name (and their continuation lines). A list of the lines is - returned, without interpretation. If the header does not occur, an - empty list is returned. If the header occurs multiple times, all - occurrences are returned. Case is not important in the header name. - """ - name = name.lower() + ':' - n = len(name) - lst = [] - hit = 0 - for line in self.headers: - if line[:n].lower() == name: - hit = 1 - elif not line[:1].isspace(): - hit = 0 - if hit: - lst.append(line) - return lst - - def getfirstmatchingheader(self, name): - """Get the first header line matching name. - - This is similar to getallmatchingheaders, but it returns only the - first matching header (and its continuation lines). - """ - name = name.lower() + ':' - n = len(name) - lst = [] - hit = 0 - for line in self.headers: - if hit: - if not line[:1].isspace(): - break - elif line[:n].lower() == name: - hit = 1 - if hit: - lst.append(line) - return lst - - def getrawheader(self, name): - """A higher-level interface to getfirstmatchingheader(). - - Return a string containing the literal text of the header but with the - keyword stripped. All leading, trailing and embedded whitespace is - kept in the string, however. Return None if the header does not - occur. - """ - - lst = self.getfirstmatchingheader(name) - if not lst: - return None - lst[0] = lst[0][len(name) + 1:] - return ''.join(lst) - - def getheader(self, name, default=None): - """Get the header value for a name. - - This is the normal interface: it returns a stripped version of the - header value for a given header name, or None if it doesn't exist. - This uses the dictionary version which finds the *last* such header. - """ - return self.dict.get(name.lower(), default) - get = getheader - - def getheaders(self, name): - """Get all values for a header. - - This returns a list of values for headers given more than once; each - value in the result list is stripped in the same way as the result of - getheader(). If the header is not given, return an empty list. - """ - result = [] - current = '' - have_header = 0 - for s in self.getallmatchingheaders(name): - if s[0].isspace(): - if current: - current = "%s\n %s" % (current, s.strip()) - else: - current = s.strip() - else: - if have_header: - result.append(current) - current = s[s.find(":") + 1:].strip() - have_header = 1 - if have_header: - result.append(current) - return result - - def getaddr(self, name): - """Get a single address from a header, as a tuple. - - An example return value: - ('Guido van Rossum', 'guido@cwi.nl') - """ - # New, by Ben Escoto - alist = self.getaddrlist(name) - if alist: - return alist[0] - else: - return (None, None) - - def getaddrlist(self, name): - """Get a list of addresses from a header. - - Retrieves a list of addresses from a header, where each address is a - tuple as returned by getaddr(). Scans all named headers, so it works - properly with multiple To: or Cc: headers for example. - """ - raw = [] - for h in self.getallmatchingheaders(name): - if h[0] in ' \t': - raw.append(h) - else: - if raw: - raw.append(', ') - i = h.find(':') - if i > 0: - addr = h[i+1:] - raw.append(addr) - alladdrs = ''.join(raw) - a = AddressList(alladdrs) - return a.addresslist - - def getdate(self, name): - """Retrieve a date field from a header. - - Retrieves a date field from the named header, returning a tuple - compatible with time.mktime(). - """ - try: - data = self[name] - except KeyError: - return None - return parsedate(data) - - def getdate_tz(self, name): - """Retrieve a date field from a header as a 10-tuple. - - The first 9 elements make up a tuple compatible with time.mktime(), - and the 10th is the offset of the poster's time zone from GMT/UTC. - """ - try: - data = self[name] - except KeyError: - return None - return parsedate_tz(data) - - - # Access as a dictionary (only finds *last* header of each type): - - def __len__(self): - """Get the number of headers in a message.""" - return len(self.dict) - - def __getitem__(self, name): - """Get a specific header, as from a dictionary.""" - return self.dict[name.lower()] - - def __setitem__(self, name, value): - """Set the value of a header. - - Note: This is not a perfect inversion of __getitem__, because any - changed headers get stuck at the end of the raw-headers list rather - than where the altered header was. - """ - del self[name] # Won't fail if it doesn't exist - self.dict[name.lower()] = value - text = name + ": " + value - for line in text.split("\n"): - self.headers.append(line + "\n") - - def __delitem__(self, name): - """Delete all occurrences of a specific header, if it is present.""" - name = name.lower() - if not name in self.dict: - return - del self.dict[name] - name = name + ':' - n = len(name) - lst = [] - hit = 0 - for i in range(len(self.headers)): - line = self.headers[i] - if line[:n].lower() == name: - hit = 1 - elif not line[:1].isspace(): - hit = 0 - if hit: - lst.append(i) - for i in reversed(lst): - del self.headers[i] - - def setdefault(self, name, default=""): - lowername = name.lower() - if lowername in self.dict: - return self.dict[lowername] - else: - text = name + ": " + default - for line in text.split("\n"): - self.headers.append(line + "\n") - self.dict[lowername] = default - return default - - def __contains__(self, name): - """Determine whether a message contains the named header.""" - return name.lower() in self.dict - - def __iter__(self): - return iter(self.dict) - - def keys(self): - """Get all of a message's header field names.""" - return list(self.dict.keys()) - - def values(self): - """Get all of a message's header field values.""" - return list(self.dict.values()) - - def items(self): - """Get all of a message's headers. - - Returns a list of name, value tuples. - """ - return list(self.dict.items()) - - def __str__(self): - return ''.join(self.headers) - - -# Utility functions -# ----------------- - -# XXX Should fix unquote() and quote() to be really conformant. -# XXX The inverses of the parse functions may also be useful. - - -def unquote(s): - """Remove quotes from a string.""" - if len(s) > 1: - if s.startswith('"') and s.endswith('"'): - return s[1:-1].replace('\\\\', '\\').replace('\\"', '"') - if s.startswith('<') and s.endswith('>'): - return s[1:-1] - return s - - -def quote(s): - """Add quotes around a string.""" - return s.replace('\\', '\\\\').replace('"', '\\"') - - -def parseaddr(address): - """Parse an address into a (realname, mailaddr) tuple.""" - a = AddressList(address) - lst = a.addresslist - if not lst: - return (None, None) - return lst[0] - - -class AddrlistClass: - """Address parser class by Ben Escoto. - - To understand what this class does, it helps to have a copy of - RFC 2822 in front of you. - - http://www.faqs.org/rfcs/rfc2822.html - - Note: this class interface is deprecated and may be removed in the future. - Use rfc822.AddressList instead. - """ - - def __init__(self, field): - """Initialize a new instance. - - `field' is an unparsed address header field, containing one or more - addresses. - """ - self.specials = '()<>@,:;.\"[]' - self.pos = 0 - self.LWS = ' \t' - self.CR = '\r\n' - self.atomends = self.specials + self.LWS + self.CR - # Note that RFC 2822 now specifies `.' as obs-phrase, meaning that it - # is obsolete syntax. RFC 2822 requires that we recognize obsolete - # syntax, so allow dots in phrases. - self.phraseends = self.atomends.replace('.', '') - self.field = field - self.commentlist = [] - - def gotonext(self): - """Parse up to the start of the next address.""" - while self.pos < len(self.field): - if self.field[self.pos] in self.LWS + '\n\r': - self.pos = self.pos + 1 - elif self.field[self.pos] == '(': - self.commentlist.append(self.getcomment()) - else: break - - def getaddrlist(self): - """Parse all addresses. - - Returns a list containing all of the addresses. - """ - result = [] - ad = self.getaddress() - while ad: - result += ad - ad = self.getaddress() - return result - - def getaddress(self): - """Parse the next address.""" - self.commentlist = [] - self.gotonext() - - oldpos = self.pos - oldcl = self.commentlist - plist = self.getphraselist() - - self.gotonext() - returnlist = [] - - if self.pos >= len(self.field): - # Bad email address technically, no domain. - if plist: - returnlist = [(' '.join(self.commentlist), plist[0])] - - elif self.field[self.pos] in '.@': - # email address is just an addrspec - # this isn't very efficient since we start over - self.pos = oldpos - self.commentlist = oldcl - addrspec = self.getaddrspec() - returnlist = [(' '.join(self.commentlist), addrspec)] - - elif self.field[self.pos] == ':': - # address is a group - returnlist = [] - - fieldlen = len(self.field) - self.pos += 1 - while self.pos < len(self.field): - self.gotonext() - if self.pos < fieldlen and self.field[self.pos] == ';': - self.pos += 1 - break - returnlist = returnlist + self.getaddress() - - elif self.field[self.pos] == '<': - # Address is a phrase then a route addr - routeaddr = self.getrouteaddr() - - if self.commentlist: - returnlist = [(' '.join(plist) + ' (' + \ - ' '.join(self.commentlist) + ')', routeaddr)] - else: returnlist = [(' '.join(plist), routeaddr)] - - else: - if plist: - returnlist = [(' '.join(self.commentlist), plist[0])] - elif self.field[self.pos] in self.specials: - self.pos += 1 - - self.gotonext() - if self.pos < len(self.field) and self.field[self.pos] == ',': - self.pos += 1 - return returnlist - - def getrouteaddr(self): - """Parse a route address (Return-path value). - - This method just skips all the route stuff and returns the addrspec. - """ - if self.field[self.pos] != '<': - return - - expectroute = 0 - self.pos += 1 - self.gotonext() - adlist = "" - while self.pos < len(self.field): - if expectroute: - self.getdomain() - expectroute = 0 - elif self.field[self.pos] == '>': - self.pos += 1 - break - elif self.field[self.pos] == '@': - self.pos += 1 - expectroute = 1 - elif self.field[self.pos] == ':': - self.pos += 1 - else: - adlist = self.getaddrspec() - self.pos += 1 - break - self.gotonext() - - return adlist - - def getaddrspec(self): - """Parse an RFC 2822 addr-spec.""" - aslist = [] - - self.gotonext() - while self.pos < len(self.field): - if self.field[self.pos] == '.': - aslist.append('.') - self.pos += 1 - elif self.field[self.pos] == '"': - aslist.append('"%s"' % self.getquote()) - elif self.field[self.pos] in self.atomends: - break - else: aslist.append(self.getatom()) - self.gotonext() - - if self.pos >= len(self.field) or self.field[self.pos] != '@': - return ''.join(aslist) - - aslist.append('@') - self.pos += 1 - self.gotonext() - return ''.join(aslist) + self.getdomain() - - def getdomain(self): - """Get the complete domain name from an address.""" - sdlist = [] - while self.pos < len(self.field): - if self.field[self.pos] in self.LWS: - self.pos += 1 - elif self.field[self.pos] == '(': - self.commentlist.append(self.getcomment()) - elif self.field[self.pos] == '[': - sdlist.append(self.getdomainliteral()) - elif self.field[self.pos] == '.': - self.pos += 1 - sdlist.append('.') - elif self.field[self.pos] in self.atomends: - break - else: sdlist.append(self.getatom()) - return ''.join(sdlist) - - def getdelimited(self, beginchar, endchars, allowcomments = 1): - """Parse a header fragment delimited by special characters. - - `beginchar' is the start character for the fragment. If self is not - looking at an instance of `beginchar' then getdelimited returns the - empty string. - - `endchars' is a sequence of allowable end-delimiting characters. - Parsing stops when one of these is encountered. - - If `allowcomments' is non-zero, embedded RFC 2822 comments are allowed - within the parsed fragment. - """ - if self.field[self.pos] != beginchar: - return '' - - slist = [''] - quote = 0 - self.pos += 1 - while self.pos < len(self.field): - if quote == 1: - slist.append(self.field[self.pos]) - quote = 0 - elif self.field[self.pos] in endchars: - self.pos += 1 - break - elif allowcomments and self.field[self.pos] == '(': - slist.append(self.getcomment()) - continue # have already advanced pos from getcomment - elif self.field[self.pos] == '\\': - quote = 1 - else: - slist.append(self.field[self.pos]) - self.pos += 1 - - return ''.join(slist) - - def getquote(self): - """Get a quote-delimited fragment from self's field.""" - return self.getdelimited('"', '"\r', 0) - - def getcomment(self): - """Get a parenthesis-delimited fragment from self's field.""" - return self.getdelimited('(', ')\r', 1) - - def getdomainliteral(self): - """Parse an RFC 2822 domain-literal.""" - return '[%s]' % self.getdelimited('[', ']\r', 0) - - def getatom(self, atomends=None): - """Parse an RFC 2822 atom. - - Optional atomends specifies a different set of end token delimiters - (the default is to use self.atomends). This is used e.g. in - getphraselist() since phrase endings must not include the `.' (which - is legal in phrases).""" - atomlist = [''] - if atomends is None: - atomends = self.atomends - - while self.pos < len(self.field): - if self.field[self.pos] in atomends: - break - else: atomlist.append(self.field[self.pos]) - self.pos += 1 - - return ''.join(atomlist) - - def getphraselist(self): - """Parse a sequence of RFC 2822 phrases. - - A phrase is a sequence of words, which are in turn either RFC 2822 - atoms or quoted-strings. Phrases are canonicalized by squeezing all - runs of continuous whitespace into one space. - """ - plist = [] - - while self.pos < len(self.field): - if self.field[self.pos] in self.LWS: - self.pos += 1 - elif self.field[self.pos] == '"': - plist.append(self.getquote()) - elif self.field[self.pos] == '(': - self.commentlist.append(self.getcomment()) - elif self.field[self.pos] in self.phraseends: - break - else: - plist.append(self.getatom(self.phraseends)) - - return plist - -class AddressList(AddrlistClass): - """An AddressList encapsulates a list of parsed RFC 2822 addresses.""" - def __init__(self, field): - AddrlistClass.__init__(self, field) - if field: - self.addresslist = self.getaddrlist() - else: - self.addresslist = [] - - def __len__(self): - return len(self.addresslist) - - def __str__(self): - return ", ".join(map(dump_address_pair, self.addresslist)) - - def __add__(self, other): - # Set union - newaddr = AddressList(None) - newaddr.addresslist = self.addresslist[:] - for x in other.addresslist: - if not x in self.addresslist: - newaddr.addresslist.append(x) - return newaddr - - def __iadd__(self, other): - # Set union, in-place - for x in other.addresslist: - if not x in self.addresslist: - self.addresslist.append(x) - return self - - def __sub__(self, other): - # Set difference - newaddr = AddressList(None) - for x in self.addresslist: - if not x in other.addresslist: - newaddr.addresslist.append(x) - return newaddr - - def __isub__(self, other): - # Set difference, in-place - for x in other.addresslist: - if x in self.addresslist: - self.addresslist.remove(x) - return self - - def __getitem__(self, index): - # Make indexing, slices, and 'in' work - return self.addresslist[index] - -def dump_address_pair(pair): - """Dump a (name, address) pair in a canonicalized form.""" - if pair[0]: - return '"' + pair[0] + '" <' + pair[1] + '>' - else: - return pair[1] - -# Parse a date field - -_monthnames = ['jan', 'feb', 'mar', 'apr', 'may', 'jun', 'jul', - 'aug', 'sep', 'oct', 'nov', 'dec', - 'january', 'february', 'march', 'april', 'may', 'june', 'july', - 'august', 'september', 'october', 'november', 'december'] -_daynames = ['mon', 'tue', 'wed', 'thu', 'fri', 'sat', 'sun'] - -# The timezone table does not include the military time zones defined -# in RFC822, other than Z. According to RFC1123, the description in -# RFC822 gets the signs wrong, so we can't rely on any such time -# zones. RFC1123 recommends that numeric timezone indicators be used -# instead of timezone names. - -_timezones = {'UT':0, 'UTC':0, 'GMT':0, 'Z':0, - 'AST': -400, 'ADT': -300, # Atlantic (used in Canada) - 'EST': -500, 'EDT': -400, # Eastern - 'CST': -600, 'CDT': -500, # Central - 'MST': -700, 'MDT': -600, # Mountain - 'PST': -800, 'PDT': -700 # Pacific - } - - -def parsedate_tz(data): - """Convert a date string to a time tuple. - - Accounts for military timezones. - """ - if not data: - return None - data = data.split() - if data[0][-1] in (',', '.') or data[0].lower() in _daynames: - # There's a dayname here. Skip it - del data[0] - else: - # no space after the "weekday,"? - i = data[0].rfind(',') - if i >= 0: - data[0] = data[0][i+1:] - if len(data) == 3: # RFC 850 date, deprecated - stuff = data[0].split('-') - if len(stuff) == 3: - data = stuff + data[1:] - if len(data) == 4: - s = data[3] - i = s.find('+') - if i > 0: - data[3:] = [s[:i], s[i+1:]] - else: - data.append('') # Dummy tz - if len(data) < 5: - return None - data = data[:5] - [dd, mm, yy, tm, tz] = data - mm = mm.lower() - if not mm in _monthnames: - dd, mm = mm, dd.lower() - if not mm in _monthnames: - return None - mm = _monthnames.index(mm)+1 - if mm > 12: mm = mm - 12 - if dd[-1] == ',': - dd = dd[:-1] - i = yy.find(':') - if i > 0: - yy, tm = tm, yy - if yy[-1] == ',': - yy = yy[:-1] - if not yy[0].isdigit(): - yy, tz = tz, yy - if tm[-1] == ',': - tm = tm[:-1] - tm = tm.split(':') - if len(tm) == 2: - [thh, tmm] = tm - tss = '0' - elif len(tm) == 3: - [thh, tmm, tss] = tm - else: - return None - try: - yy = int(yy) - dd = int(dd) - thh = int(thh) - tmm = int(tmm) - tss = int(tss) - except ValueError: - return None - tzoffset = None - tz = tz.upper() - if tz in _timezones: - tzoffset = _timezones[tz] - else: - try: - tzoffset = int(tz) - except ValueError: - pass - # Convert a timezone offset into seconds ; -0500 -> -18000 - if tzoffset: - if tzoffset < 0: - tzsign = -1 - tzoffset = -tzoffset - else: - tzsign = 1 - tzoffset = tzsign * ( (tzoffset//100)*3600 + (tzoffset % 100)*60) - return (yy, mm, dd, thh, tmm, tss, 0, 1, 0, tzoffset) - - -def parsedate(data): - """Convert a time string to a time tuple.""" - t = parsedate_tz(data) - if t is None: - return t - return t[:9] - - -def mktime_tz(data): - """Turn a 10-tuple as returned by parsedate_tz() into a UTC timestamp.""" - if data[9] is None: - # No zone info, so localtime is better assumption than GMT - return time.mktime(data[:8] + (-1,)) - else: - t = time.mktime(data[:8] + (0,)) - return t - data[9] - time.timezone - -def formatdate(timeval=None): - """Returns time format preferred for Internet standards. - - Sun, 06 Nov 1994 08:49:37 GMT ; RFC 822, updated by RFC 1123 - - According to RFC 1123, day and month names must always be in - English. If not for that, this code could use strftime(). It - can't because strftime() honors the locale and could generated - non-English names. - """ - if timeval is None: - timeval = time.time() - timeval = time.gmtime(timeval) - return "%s, %02d %s %04d %02d:%02d:%02d GMT" % ( - ("Mon", "Tue", "Wed", "Thu", "Fri", "Sat", "Sun")[timeval[6]], - timeval[2], - ("Jan", "Feb", "Mar", "Apr", "May", "Jun", - "Jul", "Aug", "Sep", "Oct", "Nov", "Dec")[timeval[1]-1], - timeval[0], timeval[3], timeval[4], timeval[5]) - - -# When used as script, run a small test program. -# The first command line argument must be a filename containing one -# message in RFC-822 format. - -if __name__ == '__main__': - import sys, os - file = os.path.join(os.environ['HOME'], 'Mail/inbox/1') - if sys.argv[1:]: file = sys.argv[1] - f = open(file, 'r') - m = Message(f) - print('From:', m.getaddr('from')) - print('To:', m.getaddrlist('to')) - print('Subject:', m.getheader('subject')) - print('Date:', m.getheader('date')) - date = m.getdate_tz('date') - tz = date[-1] - date = time.localtime(mktime_tz(date)) - if date: - print('ParsedDate:', time.asctime(date), end=' ') - hhmmss = tz - hhmm, ss = divmod(hhmmss, 60) - hh, mm = divmod(hhmm, 60) - print("%+03d%02d" % (hh, mm), end=' ') - if ss: print(".%02d" % ss, end=' ') - print() - else: - print('ParsedDate:', None) - m.rewindbody() - n = 0 - while f.readline(): - n += 1 - print('Lines:', n) - print('-'*70) - print('len =', len(m)) - if 'Date' in m: print('Date =', m['Date']) - if 'X-Nonsense' in m: pass - print('keys =', m.keys()) - print('values =', m.values()) - print('items =', m.items()) diff --git a/Lib/test/test_pyclbr.py b/Lib/test/test_pyclbr.py index 77b691d..70eea04 100644 --- a/Lib/test/test_pyclbr.py +++ b/Lib/test/test_pyclbr.py @@ -141,7 +141,6 @@ class PyclbrTest(TestCase): def test_easy(self): self.checkModule('pyclbr') self.checkModule('doctest', ignore=("TestResults", "_SpoofOut")) - self.checkModule('rfc822') self.checkModule('difflib', ignore=("Match",)) def test_decorators(self): diff --git a/Lib/test/test_rfc822.py b/Lib/test/test_rfc822.py deleted file mode 100644 index d2883e9..0000000 --- a/Lib/test/test_rfc822.py +++ /dev/null @@ -1,256 +0,0 @@ -import rfc822 -import unittest -from test import support - -try: - from io import StringIO -except ImportError: - from io import StringIO - - -class MessageTestCase(unittest.TestCase): - def create_message(self, msg): - return rfc822.Message(StringIO(msg)) - - def test_get(self): - msg = self.create_message( - 'To: "last, first" \n\ntest\n') - self.assert_(msg.get("to") == '"last, first" ') - self.assert_(msg.get("TO") == '"last, first" ') - self.assert_(msg.get("No-Such-Header") is None) - self.assert_(msg.get("No-Such-Header", "No-Such-Value") - == "No-Such-Value") - - def test_setdefault(self): - msg = self.create_message( - 'To: "last, first" \n\ntest\n') - self.assert_("New-Header" not in msg) - self.assert_(msg.setdefault("New-Header", "New-Value") == "New-Value") - self.assert_(msg.setdefault("New-Header", "Different-Value") - == "New-Value") - self.assertEqual(msg["new-header"], "New-Value") - - self.assertEqual(msg.setdefault("Another-Header"), "") - self.assertEqual(msg["another-header"], "") - - def check(self, msg, results): - """Check addresses and the date.""" - m = self.create_message(msg) - i = 0 - for n, a in m.getaddrlist('to') + m.getaddrlist('cc'): - try: - mn, ma = results[i][0], results[i][1] - except IndexError: - print('extra parsed address:', repr(n), repr(a)) - continue - i = i + 1 - self.assertEqual(mn, n, - "Un-expected name: %r != %r" % (mn, n)) - self.assertEqual(ma, a, - "Un-expected address: %r != %r" % (ma, a)) - if mn == n and ma == a: - pass - else: - print('not found:', repr(n), repr(a)) - - out = m.getdate('date') - if out: - self.assertEqual(out, - (1999, 1, 13, 23, 57, 35, 0, 1, 0), - "date conversion failed") - - - # Note: all test cases must have the same date (in various formats), - # or no date! - - def test_basic(self): - self.check( - 'Date: Wed, 13 Jan 1999 23:57:35 -0500\n' - 'From: Guido van Rossum \n' - 'To: "Guido van\n' - '\t : Rossum" \n' - 'Subject: test2\n' - '\n' - 'test2\n', - [('Guido van\n\t : Rossum', 'guido@python.org')]) - - self.check( - 'From: Barry \n' - 'Date: 13-Jan-1999 23:57:35 EST\n' - '\n' - 'test', - [('Guido: the Barbarian', 'guido@python.org'), - ('Guido: the Madman', 'guido@python.org') - ]) - - self.check( - 'To: "The monster with\n' - ' the very long name: Guido" \n' - 'Date: Wed, 13 Jan 1999 23:57:35 -0500\n' - '\n' - 'test', - [('The monster with\n the very long name: Guido', - 'guido@python.org')]) - - self.check( - 'To: "Amit J. Patel" \n' - 'CC: Mike Fletcher ,\n' - ' "\'string-sig@python.org\'" \n' - 'Cc: fooz@bat.com, bart@toof.com\n' - 'Cc: goit@lip.com\n' - 'Date: Wed, 13 Jan 1999 23:57:35 -0500\n' - '\n' - 'test', - [('Amit J. Patel', 'amitp@Theory.Stanford.EDU'), - ('Mike Fletcher', 'mfletch@vrtelecom.com'), - ("'string-sig@python.org'", 'string-sig@python.org'), - ('', 'fooz@bat.com'), - ('', 'bart@toof.com'), - ('', 'goit@lip.com'), - ]) - - self.check( - 'To: Some One \n' - 'From: Anudder Persin \n' - 'Date:\n' - '\n' - 'test', - [('Some One', 'someone@dom.ain')]) - - self.check( - 'To: person@dom.ain (User J. Person)\n\n', - [('User J. Person', 'person@dom.ain')]) - - def test_doublecomment(self): - # The RFC allows comments within comments in an email addr - self.check( - 'To: person@dom.ain ((User J. Person)), John Doe \n\n', - [('User J. Person', 'person@dom.ain'), ('John Doe', 'foo@bar.com')]) - - def test_twisted(self): - # This one is just twisted. I don't know what the proper - # result should be, but it shouldn't be to infloop, which is - # what used to happen! - self.check( - 'To: <[smtp:dd47@mail.xxx.edu]_at_hmhq@hdq-mdm1-imgout.companay.com>\n' - 'Date: Wed, 13 Jan 1999 23:57:35 -0500\n' - '\n' - 'test', - [('', ''), - ('', 'dd47@mail.xxx.edu'), - ('', '_at_hmhq@hdq-mdm1-imgout.companay.com'), - ]) - - def test_commas_in_full_name(self): - # This exercises the old commas-in-a-full-name bug, which - # should be doing the right thing in recent versions of the - # module. - self.check( - 'To: "last, first" \n' - '\n' - 'test', - [('last, first', 'userid@foo.net')]) - - def test_quoted_name(self): - self.check( - 'To: (Comment stuff) "Quoted name"@somewhere.com\n' - '\n' - 'test', - [('Comment stuff', '"Quoted name"@somewhere.com')]) - - def test_bogus_to_header(self): - self.check( - 'To: :\n' - 'Cc: goit@lip.com\n' - 'Date: Wed, 13 Jan 1999 23:57:35 -0500\n' - '\n' - 'test', - [('', 'goit@lip.com')]) - - def test_addr_ipquad(self): - self.check( - 'To: guido@[132.151.1.21]\n' - '\n' - 'foo', - [('', 'guido@[132.151.1.21]')]) - - def test_iter(self): - m = rfc822.Message(StringIO( - 'Date: Wed, 13 Jan 1999 23:57:35 -0500\n' - 'From: Guido van Rossum \n' - 'To: "Guido van\n' - '\t : Rossum" \n' - 'Subject: test2\n' - '\n' - 'test2\n' )) - self.assertEqual(sorted(m), ['date', 'from', 'subject', 'to']) - - def test_rfc2822_phrases(self): - # RFC 2822 (the update to RFC 822) specifies that dots in phrases are - # obsolete syntax, which conforming programs MUST recognize but NEVER - # generate (see $4.1 Miscellaneous obsolete tokens). This is a - # departure from RFC 822 which did not allow dots in non-quoted - # phrases. - self.check('To: User J. Person \n\n', - [('User J. Person', 'person@dom.ain')]) - - # This takes too long to add to the test suite -## def test_an_excrutiatingly_long_address_field(self): -## OBSCENELY_LONG_HEADER_MULTIPLIER = 10000 -## oneaddr = ('Person' * 10) + '@' + ('.'.join(['dom']*10)) + '.com' -## addr = ', '.join([oneaddr] * OBSCENELY_LONG_HEADER_MULTIPLIER) -## lst = rfc822.AddrlistClass(addr).getaddrlist() -## self.assertEqual(len(lst), OBSCENELY_LONG_HEADER_MULTIPLIER) - - def test_2getaddrlist(self): - eq = self.assertEqual - msg = self.create_message("""\ -To: aperson@dom.ain -Cc: bperson@dom.ain -Cc: cperson@dom.ain -Cc: dperson@dom.ain - -A test message. -""") - ccs = [('', a) for a in - ['bperson@dom.ain', 'cperson@dom.ain', 'dperson@dom.ain']] - addrs = msg.getaddrlist('cc') - addrs.sort() - eq(addrs, ccs) - # Try again, this one used to fail - addrs = msg.getaddrlist('cc') - addrs.sort() - eq(addrs, ccs) - - def test_parseaddr(self): - eq = self.assertEqual - eq(rfc822.parseaddr('<>'), ('', '')) - eq(rfc822.parseaddr('aperson@dom.ain'), ('', 'aperson@dom.ain')) - eq(rfc822.parseaddr('bperson@dom.ain (Bea A. Person)'), - ('Bea A. Person', 'bperson@dom.ain')) - eq(rfc822.parseaddr('Cynthia Person '), - ('Cynthia Person', 'cperson@dom.ain')) - - def test_quote_unquote(self): - eq = self.assertEqual - eq(rfc822.quote('foo\\wacky"name'), 'foo\\\\wacky\\"name') - eq(rfc822.unquote('"foo\\\\wacky\\"name"'), 'foo\\wacky"name') - - -def test_main(): - support.run_unittest(MessageTestCase) - - -if __name__ == "__main__": - test_main() diff --git a/Lib/test/test_urllib2.py b/Lib/test/test_urllib2.py index 6bcb17e..d6b3d57 100644 --- a/Lib/test/test_urllib2.py +++ b/Lib/test/test_urllib2.py @@ -586,7 +586,7 @@ class HandlerTests(unittest.TestCase): self.assertEqual(int(headers["Content-length"]), len(data)) def test_file(self): - import rfc822, socket + import email.utils, socket h = urllib2.FileHandler() o = h.parent = MockOpener() @@ -621,7 +621,7 @@ class HandlerTests(unittest.TestCase): finally: r.close() stats = os.stat(TESTFN) - modified = rfc822.formatdate(stats.st_mtime) + modified = email.utils.formatdate(stats.st_mtime, usegmt=True) finally: os.remove(TESTFN) self.assertEqual(data, towrite) diff --git a/Misc/NEWS b/Misc/NEWS index c07393c..c635e29 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -78,6 +78,8 @@ Extension Modules Library ------- +- rfc822 has been removed in favor of the email package. + - mimetools has been removed in favor of the email package. - Patch #2849: Remove use of rfc822 module from standard library. -- cgit v0.12