After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 593176 - winmail.dat sent in outgoing plaintext mail
winmail.dat sent in outgoing plaintext mail
Status: RESOLVED FIXED
Product: evolution-mapi
Classification: Applications
Component: Mail
0.26.x
Other Linux
: Normal normal
: ---
Assigned To: Johnny Jacob
evolution-mapi-maint
: 607107 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2009-08-26 15:00 UTC by paulk
Modified: 2010-02-08 13:03 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
set PR_SEND_RICH_INFO to false. (987 bytes, patch)
2009-08-27 11:49 UTC, Johnny Jacob
committed Details | Review
proposed ema patch (1.31 KB, patch)
2010-02-01 19:39 UTC, Milan Crha
reviewed Details | Review

Description paulk 2009-08-26 15:00:42 UTC
Ubuntu: 9.04, Evolution: 2.26.1-0ubuntu2, evolution-mapi: 0.26.0.1-0ubuntu, Exchange 2007

In outgoing mails, winmail.dat is sent as attachment although the mails are plaintext and is thus visible to non-outlook users. Does not happen via SMTP or owa.
Comment 1 Johnny Jacob 2009-08-27 11:49:27 UTC
Created attachment 141842 [details] [review]
set PR_SEND_RICH_INFO to false.

Haven't seen this happening in my test setups. 

Would you be able to try compiling evolution-mapi with the attached patch?
Comment 2 paulk 2009-08-27 13:13:59 UTC
There's no src/camel/camel-mapi-utils.c in 0.26.1 (0.26.0.1-0ubuntu deb source package from ubuntu), and i don't want to recompile half of my system for 0.27 :-) Sorry.

If it's fixed in 0.27, then you can ignore my bugreport, but in 0.26 I do have that winnmail.dat no matter what i try in evolution.
Comment 3 paulk 2009-08-27 19:35:19 UTC
I did the changes to camel-mapi-transport.c in 0.26.0.1, but that does not seem to have any effect. I know nothing about Exchange, could there be some Server setting enforcing that richtext mode for mapi?
Comment 4 Johnny Jacob 2009-08-28 03:07:08 UTC
(In reply to comment #3)
> I did the changes to camel-mapi-transport.c in 0.26.0.1, but that does not seem
> to have any effect. I know nothing about Exchange, could there be some Server
> setting enforcing that richtext mode for mapi?

Yes. Exchange Internet Mail connector (IMC) has a setting for this.

But according to the docs setting this property on the message should result in 'no winmail.dat' . But doesn't seem to be the case.
Comment 5 Johnny Jacob 2009-08-28 04:13:51 UTC
Comment on attachment 141842 [details] [review]
set PR_SEND_RICH_INFO to false.

Pushed this patch for 0.27.91.
Comment 6 Milan Crha 2010-02-01 19:17:24 UTC
*** Bug 607107 has been marked as a duplicate of this bug. ***
Comment 7 Milan Crha 2010-02-01 19:19:31 UTC
doesn't work, as stated in the bug 607107

Regarding MSDN, I found on
http://msdn.microsoft.com/en-us/library/cc842543.aspx
this:
>  In addition, MAPI defines a per-recipient property, PR_SEND_RICH_INFO
> (PidTagSendRichInfo), which governs whether or not TNEF information should
> be included in an outgoing message.

The crucial part is here "per-recipient", where the above patch is setting this on the message itself.
Comment 8 Milan Crha 2010-02-01 19:39:36 UTC
Created attachment 152756 [details] [review]
proposed ema patch

for evolution-mapi;

I'm unable to test this myself at the moment, can one of you give it a try please?
Comment 9 Johnny Jacob 2010-02-08 09:10:33 UTC
(In reply to comment #8)
> Created an attachment (id=152756) [details] [review]
> proposed ema patch
> 
> for evolution-mapi;
> 
> I'm unable to test this myself at the moment, can one of you give it a try
> please?


Milan, Patch looks harmless. I think we can commit this in master.
Comment 10 Milan Crha 2010-02-08 12:50:23 UTC
Created commit a8d21a4 in ema master (0.29.99+)

I mage a little change in the patch, I realized that I didn't increase the array size, so I also added
> +       recipient->in.ext_cValues = 8;

Please reopen if it'll not work for you. Thanks.
Comment 11 Milan Crha 2010-02-08 13:03:39 UTC
(In reply to comment #10)
> Created commit a8d21a4 in ema master (0.29.99+)

Err, 0.29.91+ it is supposed to be, I'm sorry.