Skip to content

Commit 182397d

Browse files
committed
looks like the problematic override is not necessary for tests ?
1 parent 51bd992 commit 182397d

1 file changed

Lines changed: 1 addition & 13 deletions

File tree

dd-java-agent/instrumentation/jetty-12/src/test/ee10/groovy/JettyServer.groovy

Lines changed: 1 addition & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,6 @@
11
import datadog.trace.agent.test.base.HttpServerTest
22
import datadog.trace.agent.test.base.WebsocketServer
33
import jakarta.servlet.Servlet
4-
import jakarta.servlet.ServletException
5-
import jakarta.servlet.http.HttpServletRequest
64
import jakarta.websocket.CloseReason
75
import jakarta.websocket.Endpoint
86
import jakarta.websocket.EndpointConfig
@@ -72,17 +70,7 @@ class JettyServer implements WebsocketServer {
7270
handler
7371
}
7472

75-
static errorHandler = new ErrorHandler() {
76-
@Override
77-
protected void writeErrorPage(HttpServletRequest request, Writer writer, int code,
78-
String message, boolean showStacks) throws IOException {
79-
Throwable th = (Throwable) request.getAttribute("jakarta.servlet.error.exception")
80-
message = th == null ? message : th instanceof ServletException ? th.getRootCause().message : th.message
81-
if (message) {
82-
writer.write(message)
83-
}
84-
}
85-
}
73+
static errorHandler = new ErrorHandler()
8674

8775
@Override
8876
void serverSendText(String[] messages) {

0 commit comments

Comments
 (0)