raw_sendmail creates TimedActions with invalid detail

[William Grant]

TimedAction.detail is meant to be a string, so it can be encoded by the various OOPS backends (RFC822, JSON, BSON, etc.). But lp.services.mail.sendmail.raw_sendmail sets detail to message_detail, which is an email.header.Header. This is particularly problematic for the BSON backend, because the Python bson package silently omits unknown types, leaving the action without a detail element.

[Martin Pool (mbp)]

  assigned mbp

i'll have a look but feel free to steal it

[Martin Pool (mbp)]

    # Note that raw_sendail has no tests, unit or otherwise.
:/

[Launchpad QA Bot (lpqabot)]

Fixed in stable r14372 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/14372>.

[Martin Pool (mbp)]

qa-ok: qastaging is still sending mail with this code deployed.

> # Note that raw_sendail has no tests, unit or otherwise.

fixed.