From 7ac7fe2cfecbbd93e1150df10fb4cee69c11d6db Mon Sep 17 00:00:00 2001
From: Mike Perry <mikeperry@torproject.org>
Date: Mon, 26 May 2014 14:56:28 -0700
Subject: [PATCH 1/4] Issue 3559: Use From or ReplyTo hostname in Message-ID if
 available.

I wrote this fix to avoid obviously specifying that I am using a mobile device
to reply to an email.

Others want this for ease of filtering messages from their host by Message-ID.
---
 src/com/fsck/k9/mail/internet/MimeMessage.java | 17 ++++++++++++++++-
 1 file changed, 16 insertions(+), 1 deletion(-)

diff --git a/src/com/fsck/k9/mail/internet/MimeMessage.java b/src/com/fsck/k9/mail/internet/MimeMessage.java
index d80b7bb22..c37abe02d 100644
--- a/src/com/fsck/k9/mail/internet/MimeMessage.java
+++ b/src/com/fsck/k9/mail/internet/MimeMessage.java
@@ -313,7 +313,22 @@ public class MimeMessage extends Message {
     }
 
     private String generateMessageId() {
-        return "<" + UUID.randomUUID().toString() + "@email.android.com>";
+        String hostname = null;
+        if (mFrom != null) {
+            int hostIdx = mFrom[0].getAddress().lastIndexOf("@");
+            hostname = mFrom[0].getAddress().substring(hostIdx);
+        }
+
+        if (hostname == null && mReplyTo != null) {
+            int hostIdx = mReplyTo[0].getAddress().lastIndexOf("@");
+            hostname = mReplyTo[0].getAddress().substring(hostIdx);
+        }
+
+        if (hostname != null) {
+            return "<" + UUID.randomUUID().toString() + hostname + ">";
+        } else {
+            return "<" + UUID.randomUUID().toString() + "@email.android.com>";
+        }
     }
 
     public void setMessageId(String messageId) throws UnavailableStorageException {

From 87802a01efc47be76bab7b1d89eb8c184d754995 Mon Sep 17 00:00:00 2001
From: Mike Perry <mikeperry@torproject.org>
Date: Mon, 26 May 2014 14:57:25 -0700
Subject: [PATCH 2/4] Issue 4690: Add privacy pref to omit K-9 User-Agent
 header.

---
 res/values/strings.xml                          |  1 +
 res/xml/global_preferences.xml                  |  5 +++++
 src/com/fsck/k9/K9.java                         | 10 ++++++++++
 src/com/fsck/k9/activity/MessageCompose.java    |  5 ++++-
 src/com/fsck/k9/activity/setup/Prefs.java       |  5 +++++
 src/com/fsck/k9/preferences/GlobalSettings.java |  3 +++
 6 files changed, 28 insertions(+), 1 deletion(-)

diff --git a/res/values/strings.xml b/res/values/strings.xml
index e16ca6622..3ac000e63 100644
--- a/res/values/strings.xml
+++ b/res/values/strings.xml
@@ -331,6 +331,7 @@ Please submit bug reports, contribute new features and ask questions at
     <string name="global_settings_confirm_action_spam">Spam</string>
     <string name="global_settings_confirm_action_delete_notif">Delete (from notification)</string>
 
+    <string name="global_settings_privacy_hide_useragent">Remove K-9 User-Agent from mail headers</string>
     <string name="global_settings_notification_hide_subject_title">Hide subject in notifications</string>
     <string name="global_settings_notification_hide_subject_never">Never</string>
     <string name="global_settings_notification_hide_subject_when_locked">When device is locked</string>
diff --git a/res/xml/global_preferences.xml b/res/xml/global_preferences.xml
index 90c1c7ed6..6bbc8a2cf 100644
--- a/res/xml/global_preferences.xml
+++ b/res/xml/global_preferences.xml
@@ -378,6 +378,11 @@
             android:entryValues="@array/global_settings_notification_hide_subject_values"
             android:title="@string/global_settings_notification_hide_subject_title"/>
 
+        <CheckBoxPreference
+            android:persistent="false"
+            android:key="privacy_hide_useragent"
+            android:title="@string/global_settings_privacy_hide_useragent"/>
+
     </PreferenceScreen>
 
     <PreferenceScreen
diff --git a/src/com/fsck/k9/K9.java b/src/com/fsck/k9/K9.java
index 9a841a5c7..9d2d63347 100644
--- a/src/com/fsck/k9/K9.java
+++ b/src/com/fsck/k9/K9.java
@@ -262,6 +262,7 @@ public class K9 extends Application {
     private static String mQuietTimeEnds = null;
     private static String mAttachmentDefaultPath = "";
     private static boolean mWrapFolderNames = false;
+    private static boolean mHideUserAgent = false;
 
     private static boolean useGalleryBugWorkaround = false;
     private static boolean galleryBuggy;
@@ -535,6 +536,7 @@ public class K9 extends Application {
         editor.putBoolean("messageViewReturnToList", mMessageViewReturnToList);
         editor.putBoolean("messageViewShowNext", mMessageViewShowNext);
         editor.putBoolean("wrapFolderNames", mWrapFolderNames);
+        editor.putBoolean("hideUserAgent", mHideUserAgent);
 
         editor.putString("language", language);
         editor.putInt("theme", theme.ordinal());
@@ -745,6 +747,7 @@ public class K9 extends Application {
         mMessageViewReturnToList = sprefs.getBoolean("messageViewReturnToList", false);
         mMessageViewShowNext = sprefs.getBoolean("messageViewShowNext", false);
         mWrapFolderNames = sprefs.getBoolean("wrapFolderNames", false);
+        mHideUserAgent = sprefs.getBoolean("hideUserAgent", false);
 
         useGalleryBugWorkaround = sprefs.getBoolean("useGalleryBugWorkaround", K9.isGalleryBuggy());
 
@@ -1274,6 +1277,13 @@ public class K9 extends Application {
         mWrapFolderNames = state;
     }
 
+    public static boolean hideUserAgent() {
+        return mHideUserAgent;
+    }
+    public static void setHideUserAgent(final boolean state) {
+        mHideUserAgent = state;
+    }
+
     public static String getAttachmentDefaultPath() {
         return mAttachmentDefaultPath;
     }
diff --git a/src/com/fsck/k9/activity/MessageCompose.java b/src/com/fsck/k9/activity/MessageCompose.java
index 66b07cf26..b01c875e2 100644
--- a/src/com/fsck/k9/activity/MessageCompose.java
+++ b/src/com/fsck/k9/activity/MessageCompose.java
@@ -1425,7 +1425,10 @@ public class MessageCompose extends K9Activity implements OnClickListener,
             message.setHeader("X-Confirm-Reading-To", from.toEncodedString());
             message.setHeader("Return-Receipt-To", from.toEncodedString());
         }
-        message.setHeader("User-Agent", getString(R.string.message_header_mua));
+
+        if (!K9.hideUserAgent()) {
+            message.setHeader("User-Agent", getString(R.string.message_header_mua));
+        }
 
         final String replyTo = mIdentity.getReplyTo();
         if (replyTo != null) {
diff --git a/src/com/fsck/k9/activity/setup/Prefs.java b/src/com/fsck/k9/activity/setup/Prefs.java
index fc091b1f5..dbf9188cb 100644
--- a/src/com/fsck/k9/activity/setup/Prefs.java
+++ b/src/com/fsck/k9/activity/setup/Prefs.java
@@ -85,6 +85,7 @@ public class Prefs extends K9PreferenceActivity {
     private static final String PREFERENCE_QUIET_TIME_STARTS = "quiet_time_starts";
     private static final String PREFERENCE_QUIET_TIME_ENDS = "quiet_time_ends";
     private static final String PREFERENCE_NOTIF_QUICK_DELETE = "notification_quick_delete";
+    private static final String PREFERENCE_HIDE_USERAGENT = "privacy_hide_useragent";
 
     private static final String PREFERENCE_MESSAGEVIEW_MOBILE_LAYOUT = "messageview_mobile_layout";
     private static final String PREFERENCE_AUTOFIT_WIDTH = "messageview_autofit_width";
@@ -140,6 +141,7 @@ public class Prefs extends K9PreferenceActivity {
     private CheckBoxPreference mUseGalleryBugWorkaround;
     private CheckBoxPreference mDebugLogging;
     private CheckBoxPreference mSensitiveLogging;
+    private CheckBoxPreference mHideUserAgent;
     private CheckBoxPreference mWrapFolderNames;
     private CheckBoxListPreference mVisibleRefileActions;
 
@@ -382,9 +384,11 @@ public class Prefs extends K9PreferenceActivity {
 
         mDebugLogging = (CheckBoxPreference)findPreference(PREFERENCE_DEBUG_LOGGING);
         mSensitiveLogging = (CheckBoxPreference)findPreference(PREFERENCE_SENSITIVE_LOGGING);
+        mHideUserAgent = (CheckBoxPreference)findPreference(PREFERENCE_HIDE_USERAGENT);
 
         mDebugLogging.setChecked(K9.DEBUG);
         mSensitiveLogging.setChecked(K9.DEBUG_SENSITIVE);
+        mHideUserAgent.setChecked(K9.hideUserAgent());
 
         mAttachmentPathPreference = findPreference(PREFERENCE_ATTACHMENT_DEF_PATH);
         mAttachmentPathPreference.setSummary(K9.getAttachmentDefaultPath());
@@ -532,6 +536,7 @@ public class Prefs extends K9PreferenceActivity {
         }
         K9.DEBUG = mDebugLogging.isChecked();
         K9.DEBUG_SENSITIVE = mSensitiveLogging.isChecked();
+        K9.setHideUserAgent(mHideUserAgent.isChecked());
 
         Editor editor = preferences.edit();
         K9.save(editor);
diff --git a/src/com/fsck/k9/preferences/GlobalSettings.java b/src/com/fsck/k9/preferences/GlobalSettings.java
index a55da2662..14124add9 100644
--- a/src/com/fsck/k9/preferences/GlobalSettings.java
+++ b/src/com/fsck/k9/preferences/GlobalSettings.java
@@ -249,6 +249,9 @@ public class GlobalSettings {
         s.put("fontSizeMessageViewContentPercent", Settings.versions(
                 new V(31, new IntegerRangeSetting(40, 250, 100))
             ));
+        s.put("hideUserAgent", Settings.versions(
+                new V(32, new BooleanSetting(false))
+            ));
 
         SETTINGS = Collections.unmodifiableMap(s);
 

From 38c90146e1f57f4f505b2c2ec9be33d0b0145fef Mon Sep 17 00:00:00 2001
From: Mike Perry <mikeperry@torproject.org>
Date: Mon, 26 May 2014 17:14:07 -0700
Subject: [PATCH 3/4] Issue 6372: Add preference for setting timezone to UTC.

This is a privacy preference to avoid leaking your current location while
replying to email.
---
 res/values/strings.xml                          |  1 +
 res/xml/global_preferences.xml                  |  5 +++++
 src/com/fsck/k9/K9.java                         | 10 ++++++++++
 src/com/fsck/k9/activity/setup/Prefs.java       |  5 +++++
 src/com/fsck/k9/mail/internet/MimeMessage.java  |  7 +++++++
 src/com/fsck/k9/preferences/GlobalSettings.java |  3 +++
 6 files changed, 31 insertions(+)

diff --git a/res/values/strings.xml b/res/values/strings.xml
index 3ac000e63..6e509193c 100644
--- a/res/values/strings.xml
+++ b/res/values/strings.xml
@@ -332,6 +332,7 @@ Please submit bug reports, contribute new features and ask questions at
     <string name="global_settings_confirm_action_delete_notif">Delete (from notification)</string>
 
     <string name="global_settings_privacy_hide_useragent">Remove K-9 User-Agent from mail headers</string>
+    <string name="global_settings_privacy_hide_timezone">Use UTC as time zone in mail headers</string>
     <string name="global_settings_notification_hide_subject_title">Hide subject in notifications</string>
     <string name="global_settings_notification_hide_subject_never">Never</string>
     <string name="global_settings_notification_hide_subject_when_locked">When device is locked</string>
diff --git a/res/xml/global_preferences.xml b/res/xml/global_preferences.xml
index 6bbc8a2cf..c985c35d9 100644
--- a/res/xml/global_preferences.xml
+++ b/res/xml/global_preferences.xml
@@ -383,6 +383,11 @@
             android:key="privacy_hide_useragent"
             android:title="@string/global_settings_privacy_hide_useragent"/>
 
+        <CheckBoxPreference
+            android:persistent="false"
+            android:key="privacy_hide_timezone"
+            android:title="@string/global_settings_privacy_hide_timezone"/>
+
     </PreferenceScreen>
 
     <PreferenceScreen
diff --git a/src/com/fsck/k9/K9.java b/src/com/fsck/k9/K9.java
index 9d2d63347..0c320bafd 100644
--- a/src/com/fsck/k9/K9.java
+++ b/src/com/fsck/k9/K9.java
@@ -263,6 +263,7 @@ public class K9 extends Application {
     private static String mAttachmentDefaultPath = "";
     private static boolean mWrapFolderNames = false;
     private static boolean mHideUserAgent = false;
+    private static boolean mHideTimeZone = false;
 
     private static boolean useGalleryBugWorkaround = false;
     private static boolean galleryBuggy;
@@ -537,6 +538,7 @@ public class K9 extends Application {
         editor.putBoolean("messageViewShowNext", mMessageViewShowNext);
         editor.putBoolean("wrapFolderNames", mWrapFolderNames);
         editor.putBoolean("hideUserAgent", mHideUserAgent);
+        editor.putBoolean("hideTimeZone", mHideTimeZone);
 
         editor.putString("language", language);
         editor.putInt("theme", theme.ordinal());
@@ -748,6 +750,7 @@ public class K9 extends Application {
         mMessageViewShowNext = sprefs.getBoolean("messageViewShowNext", false);
         mWrapFolderNames = sprefs.getBoolean("wrapFolderNames", false);
         mHideUserAgent = sprefs.getBoolean("hideUserAgent", false);
+        mHideTimeZone = sprefs.getBoolean("hideTimeZone", false);
 
         useGalleryBugWorkaround = sprefs.getBoolean("useGalleryBugWorkaround", K9.isGalleryBuggy());
 
@@ -1284,6 +1287,13 @@ public class K9 extends Application {
         mHideUserAgent = state;
     }
 
+    public static boolean hideTimeZone() {
+        return mHideTimeZone;
+    }
+    public static void setHideTimeZone(final boolean state) {
+        mHideTimeZone = state;
+    }
+
     public static String getAttachmentDefaultPath() {
         return mAttachmentDefaultPath;
     }
diff --git a/src/com/fsck/k9/activity/setup/Prefs.java b/src/com/fsck/k9/activity/setup/Prefs.java
index dbf9188cb..b427a2031 100644
--- a/src/com/fsck/k9/activity/setup/Prefs.java
+++ b/src/com/fsck/k9/activity/setup/Prefs.java
@@ -86,6 +86,7 @@ public class Prefs extends K9PreferenceActivity {
     private static final String PREFERENCE_QUIET_TIME_ENDS = "quiet_time_ends";
     private static final String PREFERENCE_NOTIF_QUICK_DELETE = "notification_quick_delete";
     private static final String PREFERENCE_HIDE_USERAGENT = "privacy_hide_useragent";
+    private static final String PREFERENCE_HIDE_TIMEZONE = "privacy_hide_timezone";
 
     private static final String PREFERENCE_MESSAGEVIEW_MOBILE_LAYOUT = "messageview_mobile_layout";
     private static final String PREFERENCE_AUTOFIT_WIDTH = "messageview_autofit_width";
@@ -142,6 +143,7 @@ public class Prefs extends K9PreferenceActivity {
     private CheckBoxPreference mDebugLogging;
     private CheckBoxPreference mSensitiveLogging;
     private CheckBoxPreference mHideUserAgent;
+    private CheckBoxPreference mHideTimeZone;
     private CheckBoxPreference mWrapFolderNames;
     private CheckBoxListPreference mVisibleRefileActions;
 
@@ -385,10 +387,12 @@ public class Prefs extends K9PreferenceActivity {
         mDebugLogging = (CheckBoxPreference)findPreference(PREFERENCE_DEBUG_LOGGING);
         mSensitiveLogging = (CheckBoxPreference)findPreference(PREFERENCE_SENSITIVE_LOGGING);
         mHideUserAgent = (CheckBoxPreference)findPreference(PREFERENCE_HIDE_USERAGENT);
+        mHideTimeZone = (CheckBoxPreference)findPreference(PREFERENCE_HIDE_TIMEZONE);
 
         mDebugLogging.setChecked(K9.DEBUG);
         mSensitiveLogging.setChecked(K9.DEBUG_SENSITIVE);
         mHideUserAgent.setChecked(K9.hideUserAgent());
+        mHideTimeZone.setChecked(K9.hideTimeZone());
 
         mAttachmentPathPreference = findPreference(PREFERENCE_ATTACHMENT_DEF_PATH);
         mAttachmentPathPreference.setSummary(K9.getAttachmentDefaultPath());
@@ -537,6 +541,7 @@ public class Prefs extends K9PreferenceActivity {
         K9.DEBUG = mDebugLogging.isChecked();
         K9.DEBUG_SENSITIVE = mSensitiveLogging.isChecked();
         K9.setHideUserAgent(mHideUserAgent.isChecked());
+        K9.setHideTimeZone(mHideTimeZone.isChecked());
 
         Editor editor = preferences.edit();
         K9.save(editor);
diff --git a/src/com/fsck/k9/mail/internet/MimeMessage.java b/src/com/fsck/k9/mail/internet/MimeMessage.java
index c37abe02d..7c6fe6b2c 100644
--- a/src/com/fsck/k9/mail/internet/MimeMessage.java
+++ b/src/com/fsck/k9/mail/internet/MimeMessage.java
@@ -8,6 +8,7 @@ import java.io.OutputStream;
 import java.io.OutputStreamWriter;
 import java.text.SimpleDateFormat;
 import java.util.Date;
+import java.util.TimeZone;
 import java.util.LinkedList;
 import java.util.Locale;
 import java.util.Set;
@@ -33,6 +34,7 @@ import com.fsck.k9.mail.MessagingException;
 import com.fsck.k9.mail.Multipart;
 import com.fsck.k9.mail.Part;
 import com.fsck.k9.mail.store.UnavailableStorageException;
+import com.fsck.k9.K9;
 
 /**
  * An implementation of Message that stores all of it's metadata in RFC 822 and
@@ -150,6 +152,11 @@ public class MimeMessage extends Message {
         if (mDateFormat == null) {
             mDateFormat = new SimpleDateFormat("EEE, dd MMM yyyy HH:mm:ss Z", Locale.US);
         }
+
+        if (K9.hideTimeZone()) {
+            mDateFormat.setTimeZone(TimeZone.getTimeZone("UTC"));
+        }
+
         addHeader("Date", mDateFormat.format(sentDate));
         setInternalSentDate(sentDate);
     }
diff --git a/src/com/fsck/k9/preferences/GlobalSettings.java b/src/com/fsck/k9/preferences/GlobalSettings.java
index 14124add9..4f1d089c0 100644
--- a/src/com/fsck/k9/preferences/GlobalSettings.java
+++ b/src/com/fsck/k9/preferences/GlobalSettings.java
@@ -252,6 +252,9 @@ public class GlobalSettings {
         s.put("hideUserAgent", Settings.versions(
                 new V(32, new BooleanSetting(false))
             ));
+        s.put("hideTimeZone", Settings.versions(
+                new V(32, new BooleanSetting(false))
+            ));
 
         SETTINGS = Collections.unmodifiableMap(s);
 

From d67a59b77fd84a1f9c5b332b8f6d9f9bbd2d7a0a Mon Sep 17 00:00:00 2001
From: Mike Perry <mikeperry@torproject.org>
Date: Fri, 13 Jun 2014 17:38:48 -0700
Subject: [PATCH 4/4] Fixes for cketti's code review on pull req #472

Leave the hostname == null checks so we can fall back if a hostname is not
found. Also convert message-id to upper case to match Apple Mail (for
privacy).
---
 src/com/fsck/k9/mail/Address.java             | 10 ++++++++++
 .../fsck/k9/mail/internet/MimeMessage.java    | 20 +++++++++----------
 src/com/fsck/k9/preferences/Settings.java     |  2 +-
 3 files changed, 21 insertions(+), 11 deletions(-)

diff --git a/src/com/fsck/k9/mail/Address.java b/src/com/fsck/k9/mail/Address.java
index b49f425c0..a8ef1ee0c 100644
--- a/src/com/fsck/k9/mail/Address.java
+++ b/src/com/fsck/k9/mail/Address.java
@@ -93,6 +93,16 @@ public class Address {
         return mAddress;
     }
 
+    public String getHostname() {
+        int hostIdx = mAddress.lastIndexOf("@");
+
+        if (hostIdx == -1) {
+            return null;
+        }
+
+        return mAddress.substring(hostIdx+1);
+    }
+
     public void setAddress(String address) {
         this.mAddress = address;
     }
diff --git a/src/com/fsck/k9/mail/internet/MimeMessage.java b/src/com/fsck/k9/mail/internet/MimeMessage.java
index 7c6fe6b2c..2f69cba39 100644
--- a/src/com/fsck/k9/mail/internet/MimeMessage.java
+++ b/src/com/fsck/k9/mail/internet/MimeMessage.java
@@ -321,21 +321,21 @@ public class MimeMessage extends Message {
 
     private String generateMessageId() {
         String hostname = null;
-        if (mFrom != null) {
-            int hostIdx = mFrom[0].getAddress().lastIndexOf("@");
-            hostname = mFrom[0].getAddress().substring(hostIdx);
+
+        if (mFrom != null && mFrom.length >= 1) {
+            hostname = mFrom[0].getHostname();
         }
 
-        if (hostname == null && mReplyTo != null) {
-            int hostIdx = mReplyTo[0].getAddress().lastIndexOf("@");
-            hostname = mReplyTo[0].getAddress().substring(hostIdx);
+        if (hostname == null && mReplyTo != null && mReplyTo.length >= 1) {
+            hostname = mReplyTo[0].getHostname();
         }
 
-        if (hostname != null) {
-            return "<" + UUID.randomUUID().toString() + hostname + ">";
-        } else {
-            return "<" + UUID.randomUUID().toString() + "@email.android.com>";
+        if (hostname == null) {
+            hostname = "email.android.com";
         }
+
+        /* We use upper case here to match Apple Mail Message-ID format (for privacy) */
+        return "<" + UUID.randomUUID().toString().toUpperCase(Locale.US) + "@" + hostname + ">";
     }
 
     public void setMessageId(String messageId) throws UnavailableStorageException {
diff --git a/src/com/fsck/k9/preferences/Settings.java b/src/com/fsck/k9/preferences/Settings.java
index 11ecc921a..b4dd8ef41 100644
--- a/src/com/fsck/k9/preferences/Settings.java
+++ b/src/com/fsck/k9/preferences/Settings.java
@@ -35,7 +35,7 @@ public class Settings {
      *
      * @see SettingsExporter
      */
-    public static final int VERSION = 31;
+    public static final int VERSION = 32;
 
     public static Map<String, Object> validate(int version, Map<String,
             TreeMap<Integer, SettingsDescription>> settings,