Re: [Haskell-cafe] Proposed change to mime-mail

2011-11-28 Thread Michael Snoyman
On Wed, Nov 23, 2011 at 9:39 PM, Erik Hesselink hessel...@gmail.com wrote:
 Hi all,

 I've found and solved a problem with mime-mail, and Michael Snoyman
 asked me to send a request for feedback to -cafe, so here goes.

 In short, the issue is with address headers containing 'special'
 (non-ascii) characters. In mime-mail, these are automatically encoded
 according to RFC 2047. However, the email address part of an address
 header is not allowed to be encoded.

 My patch adds 'from', 'to', 'cc' and 'bcc' fields to the Mail data
 type, and creates correct headers from these fields. This does make
 the Mail type a bit larger, so I've also added a smart constructor
 with initializes these fields with empty lists (only 'from' is
 required).

 For more details, see my initial bug report [1] and my patches in the
 pull request [2].

 If you have any comments on this change, please let me know.

 Regards,

 Erik

 [1] https://github.com/snoyberg/mime-mail/issues/5
 [2] https://github.com/snoyberg/mime-mail/pull/6

 ___
 Haskell-Cafe mailing list
 Haskell-Cafe@haskell.org
 http://www.haskell.org/mailman/listinfo/haskell-cafe


Since there were no objections, I've merged the patches and released
mime-mail 0.4. Thanks Erik!

Michael

___
Haskell-Cafe mailing list
Haskell-Cafe@haskell.org
http://www.haskell.org/mailman/listinfo/haskell-cafe


[Haskell-cafe] Proposed change to mime-mail

2011-11-23 Thread Erik Hesselink
Hi all,

I've found and solved a problem with mime-mail, and Michael Snoyman
asked me to send a request for feedback to -cafe, so here goes.

In short, the issue is with address headers containing 'special'
(non-ascii) characters. In mime-mail, these are automatically encoded
according to RFC 2047. However, the email address part of an address
header is not allowed to be encoded.

My patch adds 'from', 'to', 'cc' and 'bcc' fields to the Mail data
type, and creates correct headers from these fields. This does make
the Mail type a bit larger, so I've also added a smart constructor
with initializes these fields with empty lists (only 'from' is
required).

For more details, see my initial bug report [1] and my patches in the
pull request [2].

If you have any comments on this change, please let me know.

Regards,

Erik

[1] https://github.com/snoyberg/mime-mail/issues/5
[2] https://github.com/snoyberg/mime-mail/pull/6

___
Haskell-Cafe mailing list
Haskell-Cafe@haskell.org
http://www.haskell.org/mailman/listinfo/haskell-cafe


Re: [Haskell-cafe] Proposed change to mime-mail

2011-11-23 Thread Erik de Castro Lopo
Erik Hesselink wrote:

 Hi all,
 
 I've found and solved a problem with mime-mail, and Michael Snoyman
 asked me to send a request for feedback to -cafe, so here goes.
 
 In short, the issue is with address headers containing 'special'
 (non-ascii) characters. In mime-mail, these are automatically encoded
 according to RFC 2047. However, the email address part of an address
 header is not allowed to be encoded.
 
 My patch adds 'from', 'to', 'cc' and 'bcc' fields to the Mail data
 type, and creates correct headers from these fields. This does make
 the Mail type a bit larger, so I've also added a smart constructor
 with initializes these fields with empty lists (only 'from' is
 required).
 
 For more details, see my initial bug report [1] and my patches in the
 pull request [2].
 
 If you have any comments on this change, please let me know.

+1

Erik
-- 
--
Erik de Castro Lopo
http://www.mega-nerd.com/

___
Haskell-Cafe mailing list
Haskell-Cafe@haskell.org
http://www.haskell.org/mailman/listinfo/haskell-cafe