Skip to content

Commit 5a48a06

Browse files
committed
html encode log entries in AppLog.toHtml()
1 parent c8a4d89 commit 5a48a06

File tree

1 file changed

+5
-8
lines changed
  • WordPressUtils/src/main/java/org/wordpress/android/util

1 file changed

+5
-8
lines changed

WordPressUtils/src/main/java/org/wordpress/android/util/AppLog.java

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -60,13 +60,13 @@ public static void e(T tag, String message) {
6060
public static void e(T tag, String message, Throwable tr) {
6161
Log.e(TAG + "-" + tag.toString(), message, tr);
6262
addEntry(tag, LogLevel.e, message + " - exception: " + tr.getMessage());
63-
addEntry(tag, LogLevel.e, "StackTrace: " + getHTMLStringStackTrace(tr));
63+
addEntry(tag, LogLevel.e, "StackTrace: " + getStringStackTrace(tr));
6464
}
6565

6666
public static void e(T tag, Throwable tr) {
6767
Log.e(TAG + "-" + tag.toString(), tr.getMessage(), tr);
6868
addEntry(tag, LogLevel.e, tr.getMessage());
69-
addEntry(tag, LogLevel.e, "StackTrace: " + getHTMLStringStackTrace(tr));
69+
addEntry(tag, LogLevel.e, "StackTrace: " + getStringStackTrace(tr));
7070
}
7171

7272
public static void e(T tag, String volleyErrorMsg, int statusCode) {
@@ -113,15 +113,15 @@ private static class LogEntry {
113113

114114
private String toHtml() {
115115
StringBuilder sb = new StringBuilder()
116-
.append("<font color='")
116+
.append("<font color=\"")
117117
.append(logLevel.toHtmlColor())
118-
.append("'>")
118+
.append("\">")
119119
.append("[")
120120
.append(logTag.name())
121121
.append("] ")
122122
.append(logLevel.name())
123123
.append(": ")
124-
.append(logText)
124+
.append(TextUtils.htmlEncode(logText).replace("\n", "<br />"))
125125
.append("</font>");
126126
return sb.toString();
127127
}
@@ -164,9 +164,6 @@ private static String getStringStackTrace(Throwable throwable) {
164164
return errors.toString();
165165
}
166166

167-
private static String getHTMLStringStackTrace(Throwable throwable) {
168-
return getStringStackTrace(throwable).replace("\n", "<br/>");
169-
}
170167

171168
/*
172169
* returns entire log as html for display (see AppLogViewerActivity)

0 commit comments

Comments
 (0)