[Buildbot-commits] [Buildbot] #2373: Exception caught notifying <buildbot.status.mail.MailNotifier

[Buildbot]

#2373: Exception caught notifying <buildbot.status.mail.MailNotifier
---------------------------+--------------------
Reporter:  materialdreams  |       Owner:
    Type:  defect          |      Status:  new
Priority:  major           |   Milestone:  0.8.+
 Version:  0.8.6p1         |  Resolution:
Keywords:  mail, encoding  |
---------------------------+--------------------
Changes (by dustin):

 * keywords:  exception buildbot.status.mail.MailNotifier => mail, encoding
 * milestone:  undecided => 0.8.+


Comment:

 The problem here is that log output doesn't have an encoding.  The
 !MailNotifier assumes utf-8, but that guess seems wrong in this case.

 If you know what the encoding is, you can set it where {{{LOG_ENCODING}}}
 is set near the top of mail.py.  Otherwise, you can change line 584 to
 read
 {{{
             text = text.decode(LOG_ENCODING, errors='replace')
 }}}

 Maybe we should do the latter by default?  Ideally the logs would get an
 encoding when they come *in* to buildbot, rather than being stored as
 bytestrings :(

-- 
Ticket URL: <http://trac.buildbot.net/ticket/2373#comment:1>
Buildbot <http://buildbot.net/>
Buildbot: build/test automation