← Back to team overview

vm team mailing list archive

[Merge] lp:~akwm/vm/digest-explicit-header into lp:vm

 

Arik has proposed merging lp:~akwm/vm/digest-explicit-header into lp:vm.

Requested reviews:
  VM development team (vm)
Related bugs:
  #612111 digest message improperly handles boundary
  https://bugs.launchpad.net/bugs/612111


since the problem with digest messages was with other clients, explicitly setting the content type to the default type will assist mail clients (gmail being one) to properly display digest messages. The default for multipart/digest messages is message/rfc822 where, it seems, clients might assume the non-specific multipar/{whatever} of text/plain
-- 
https://code.launchpad.net/~akwm/vm/digest-explicit-header/+merge/33525
Your team VM development team is requested to review the proposed merge of lp:~akwm/vm/digest-explicit-header into lp:vm.
=== modified file 'lisp/vm-digest.el'
--- lisp/vm-digest.el	2010-08-19 19:05:20 +0000
+++ lisp/vm-digest.el	2010-08-24 13:38:39 +0000
@@ -132,7 +132,8 @@
 	    (insert "\n--" boundary "--\n")
 	    (while boundary-positions
 	      (goto-char (car boundary-positions))
-	      (insert "\n--" boundary "\n\n")
+	      (insert "\n--" boundary "\n")
+	      (insert "Content-Type: message/rfc822\n\n")
 	      (setq boundary-positions (cdr boundary-positions)))
 	    (goto-char start)
 	    (setq n (length message-list))


Follow ups