This is an automated email from the ASF dual-hosted git repository.
remm pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/main by this push:
new 0dc1d2a3af Cleanup connector names in json
0dc1d2a3af is described below
commit 0dc1d2a3afb4ea06e6d2c1efd94387111534e654
Author: remm <r...@apache.org>
AuthorDate: Fri Jan 5 16:17:06 2024 +0100
Cleanup connector names in json
---
.../apache/catalina/manager/StatusTransformer.java | 19 ++++++++++++-------
.../catalina/manager/TestStatusTransformer.java | 3 +--
2 files changed, 13 insertions(+), 9 deletions(-)
diff --git a/java/org/apache/catalina/manager/StatusTransformer.java
b/java/org/apache/catalina/manager/StatusTransformer.java
index f06c172ab4..d6719d70b9 100644
--- a/java/org/apache/catalina/manager/StatusTransformer.java
+++ b/java/org/apache/catalina/manager/StatusTransformer.java
@@ -238,7 +238,8 @@ public class StatusTransformer {
appendJSonValue(writer, "usageUsed",
Long.toString(usage.getUsed()));
writer.append('}');
}
- writer.append(']').println();
+ writer.println();
+ indent(writer, 2).append(']').println();
indent(writer, 1).append('}');
}
@@ -437,11 +438,15 @@ public class StatusTransformer {
writer.write("</connector>");
} else if (mode == 2) {
- indent(writer, 1).append('{').println();
+ indent(writer, 2).append('{').println();
- indent(writer, 2);
- appendJSonValue(writer, "name",
JSONFilter.escape(name)).append(',').println();
- indent(writer,
2).append('"').append("threadInfo").append('"').append(':').append('{');
+ indent(writer, 3);
+ String jsonName = JSONFilter.escape(name);
+ if (jsonName.length() > 4 && jsonName.startsWith("\\\"") &&
jsonName.endsWith("\\\"")) {
+ jsonName = jsonName.substring(2, jsonName.length() - 2);
+ }
+ appendJSonValue(writer, "name", jsonName).append(',').println();
+ indent(writer,
3).append('"').append("threadInfo").append('"').append(':').append('{');
appendJSonValue(writer, "maxThreads",
mBeanServer.getAttribute(tpName, "maxThreads").toString()).append(',');
appendJSonValue(writer, "currentThreadCount",
mBeanServer.getAttribute(tpName, "currentThreadCount").toString()).append(',');
appendJSonValue(writer, "currentThreadsBusy",
mBeanServer.getAttribute(tpName, "currentThreadsBusy").toString());
@@ -457,7 +462,7 @@ public class StatusTransformer {
if (grpName != null) {
writer.append(',').println();
- indent(writer,
2).append('"').append("requestInfo").append('"').append(':').append('{');
+ indent(writer,
3).append('"').append("requestInfo").append('"').append(':').append('{');
appendJSonValue(writer, "maxTime",
mBeanServer.getAttribute(grpName, "maxTime").toString()).append(',');
appendJSonValue(writer, "processingTime",
mBeanServer.getAttribute(grpName, "processingTime").toString()).append(',');
appendJSonValue(writer, "requestCount",
mBeanServer.getAttribute(grpName, "requestCount").toString()).append(',');
@@ -468,7 +473,7 @@ public class StatusTransformer {
// Note: No detailed per processor info
}
- indent(writer, 1).append('}');
+ indent(writer, 2).append('}');
}
}
diff --git a/test/org/apache/catalina/manager/TestStatusTransformer.java
b/test/org/apache/catalina/manager/TestStatusTransformer.java
index b031004979..3cd772fdd4 100644
--- a/test/org/apache/catalina/manager/TestStatusTransformer.java
+++ b/test/org/apache/catalina/manager/TestStatusTransformer.java
@@ -72,10 +72,9 @@ public class TestStatusTransformer extends TomcatBaseTest {
client.connect();
client.processRequest(true);
String json = client.getResponseBody();
-
+ System.out.println(json);
JSONParser parser = new JSONParser(json);
String result = parser.parse().toString();
- System.out.println(result);
Assert.assertTrue(result.contains("name=localhost/"));
}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
