Skip to content

Commit 8102cec

Browse files
committed
address copilot review
1 parent 3f71147 commit 8102cec

1 file changed

Lines changed: 14 additions & 10 deletions

File tree

driver-core/src/test/unit/com/mongodb/internal/diagnostics/ThreadDumpOnFailureExtension.java

Lines changed: 14 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@
3333
*/
3434
public final class ThreadDumpOnFailureExtension implements TestWatcher {
3535

36-
private static final Logger LOGGER = Loggers.getLogger(ThreadDumpOnFailureExtension.class.getName());
36+
private static final Logger LOGGER = Loggers.getLogger(ThreadDumpOnFailureExtension.class.getSimpleName());
3737

3838
@Override
3939
public void testFailed(final ExtensionContext context, final Throwable cause) {
@@ -43,15 +43,19 @@ public void testFailed(final ExtensionContext context, final Throwable cause) {
4343
}
4444

4545
private static String getAllThreadsDump() {
46-
final ThreadMXBean threadMXBean = ManagementFactory.getThreadMXBean();
47-
ThreadInfo[] threadInfos = threadMXBean.dumpAllThreads(
48-
threadMXBean.isObjectMonitorUsageSupported(),
49-
threadMXBean.isSynchronizerUsageSupported()
50-
);
51-
StringBuilder sb = new StringBuilder(1024);
52-
for (ThreadInfo info : threadInfos) {
53-
sb.append(info);
46+
try {
47+
final ThreadMXBean threadMXBean = ManagementFactory.getThreadMXBean();
48+
ThreadInfo[] threadInfos = threadMXBean.dumpAllThreads(
49+
threadMXBean.isObjectMonitorUsageSupported(),
50+
threadMXBean.isSynchronizerUsageSupported()
51+
);
52+
StringBuilder sb = new StringBuilder(1024);
53+
for (ThreadInfo info : threadInfos) {
54+
sb.append(info);
55+
}
56+
return sb.toString();
57+
} catch (final SecurityException exc) {
58+
return "Unable to get thread dump due to security manager restrictions: " + exc.getMessage();
5459
}
55-
return sb.toString();
5660
}
5761
}

0 commit comments

Comments
 (0)