Skip to content

Commit 7780dbe

Browse files
author
Oleg Aleksandrov
committed
Issue-1160 Code clean up, send to client only error type without whole error message
1 parent a97965d commit 7780dbe

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

src/main/java/org/java_websocket/WebSocketImpl.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,6 @@
2626
package org.java_websocket;
2727

2828
import java.io.IOException;
29-
import java.io.PrintWriter;
30-
import java.io.StringWriter;
3129
import java.net.InetSocketAddress;
3230
import java.nio.ByteBuffer;
3331
import java.nio.channels.ByteChannel;
@@ -419,7 +417,8 @@ private void decodeFrames(ByteBuffer socketBuffer) {
419417
log.error("Closing web socket due to an error during frame processing");
420418
Exception exception = new Exception(e);
421419
wsl.onWebsocketError(this, exception);
422-
close(CloseFrame.UNEXPECTED_CONDITION, exception.getMessage());
420+
String errorMessage = "Got error " + e.getClass().getName() + " on the server side";
421+
close(CloseFrame.UNEXPECTED_CONDITION, errorMessage);
423422
}
424423
}
425424

src/main/java/org/java_websocket/server/WebSocketServer.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1083,7 +1083,7 @@ public void run() {
10831083
if (ws != null) {
10841084
ws.close();
10851085
}
1086-
log.error("Got fatal error in worker thread" + getName());
1086+
log.error("Got fatal error in worker thread {}", getName());
10871087
Exception exception = new Exception(e);
10881088
handleFatal(ws, exception);
10891089
} catch (Throwable e) {

0 commit comments

Comments
 (0)