bug: change default UA agent string for MMS.
- add conditional logging for UAProf url.
- if the MMS port isn't set, don't use Log.e() to log it.
Change-Id: I9557763d209226eafb5cb3b0b7e4092287f5ef29
diff --git a/src/com/android/mms/MmsConfig.java b/src/com/android/mms/MmsConfig.java
index 6bd2844..93a9f8b 100644
--- a/src/com/android/mms/MmsConfig.java
+++ b/src/com/android/mms/MmsConfig.java
@@ -34,7 +34,7 @@
private static final boolean LOCAL_LOGV = DEBUG ? Config.LOGD : Config.LOGV;
private static final String DEFAULT_HTTP_KEY_X_WAP_PROFILE = "x-wap-profile";
- private static final String DEFAULT_USER_AGENT = "Android-Mms/0.1";
+ private static final String DEFAULT_USER_AGENT = "Android-Mms/2.0";
private static final int MAX_IMAGE_HEIGHT = 480;
private static final int MAX_IMAGE_WIDTH = 640;
diff --git a/src/com/android/mms/transaction/HttpUtils.java b/src/com/android/mms/transaction/HttpUtils.java
index 0514ad6..1448426 100644
--- a/src/com/android/mms/transaction/HttpUtils.java
+++ b/src/com/android/mms/transaction/HttpUtils.java
@@ -157,6 +157,10 @@
String xWapProfileUrl = MmsConfig.getUaProfUrl();
if (xWapProfileUrl != null) {
+ if (Log.isLoggable(LogTag.TRANSACTION, Log.VERBOSE)) {
+ Log.d(LogTag.TRANSACTION,
+ "[HttpUtils] httpConn: xWapProfUrl=" + xWapProfileUrl);
+ }
req.addHeader(xWapProfileTagName, xWapProfileUrl);
}
}
diff --git a/src/com/android/mms/transaction/TransactionSettings.java b/src/com/android/mms/transaction/TransactionSettings.java
index b4848da..59a2148 100644
--- a/src/com/android/mms/transaction/TransactionSettings.java
+++ b/src/com/android/mms/transaction/TransactionSettings.java
@@ -85,7 +85,11 @@
try {
mProxyPort = Integer.parseInt(portString);
} catch (NumberFormatException e) {
- Log.e(TAG, "Bad port number format: " + portString, e);
+ if (TextUtils.isEmpty(portString)) {
+ Log.w(TAG, "mms port not set!");
+ } else {
+ Log.e(TAG, "Bad port number format: " + portString, e);
+ }
}
}
}