Add analytics to track of what cipher suites are being used

b/18001842
I do this in MailTransport, which is only used by POP and IMAP.
I can't log Analytics in SSLSocketFactoryWrapper, because that
is in emailcommon and would cause a circular dependancy between
emailcomman and UnifiedEmail.
This will not yet work for Exchange, because it gets its socket
indirectly through EmailClientConnectionManager, using a
SchemeRegistry. Still, it will be helpful to get this info for POP
and IMAP.

Change-Id: Ibb9d22c83c49d0ba5090295b5321ca6afb09e65b

1 files changed, 1 insertions, 1 deletions

diff --git a/emailcommon/src/com/android/emailcommon/utility/SSLSocketFactoryWrapper.java b/emailcommon/src/com/android/emailcommon/utility/SSLSocketFactoryWrapper.java
index 516084fe4..66b596bff 100644
--- a/emailcommon/src/com/android/emailcommon/utility/SSLSocketFactoryWrapper.java
+++ b/emailcommon/src/com/android/emailcommon/utility/SSLSocketFactoryWrapper.java
@@ -234,10 +234,10 @@ public class SSLSocketFactoryWrapper extends javax.net.ssl.SSLSocketFactory {
         ssl.startHandshake();
 
         SSLSession session = ssl.getSession();
-        LogUtils.d(LogUtils.TAG, "using cipherSuite %s", session.getCipherSuite());
         if (session == null) {
             throw new SSLException("Cannot verify SSL socket without session");
         }
+        LogUtils.d(LogUtils.TAG, "using cipherSuite %s", session.getCipherSuite());
         if (!HttpsURLConnection.getDefaultHostnameVerifier().verify(hostname, session)) {
             throw new SSLPeerUnverifiedException("Cannot verify hostname: " + hostname);
         }