j62
/
ctbrec
2
2
Fork 2
Code Issues 5 Pull Requests Packages Projects Releases 3 Wiki Activity

On error write a log of the recent received strings 

When the websocket message parser throws an exception,
write a logfile with the 100 last recent strings, so
that we can debug, what happened.
This commit is contained in:
0xboobface 2018-10-24 16:46:30 +02:00
parent ea038ae06b
commit b6a31a258e
1 changed files with 32 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

43
src/main/java/ctbrec/sites/mfc/MyFreeCamsClient.java
View File

@ -2,6 +2,7 @@ package ctbrec.sites.mfc;
import static ctbrec.sites.mfc.MessageTypes.*; import static ctbrec.sites.mfc.MessageTypes.*;
import java.io.FileOutputStream;
import java.io.IOException; import java.io.IOException;
import java.io.UnsupportedEncodingException; import java.io.UnsupportedEncodingException;
import java.net.URLDecoder; import java.net.URLDecoder;
@ -21,6 +22,7 @@ import org.json.JSONObject;
import org.slf4j.Logger; import org.slf4j.Logger;
import org.slf4j.LoggerFactory; import org.slf4j.LoggerFactory;
import com.google.common.collect.EvictingQueue;
import com.squareup.moshi.JsonAdapter; import com.squareup.moshi.JsonAdapter;
import com.squareup.moshi.Moshi; import com.squareup.moshi.Moshi;
@ -51,6 +53,8 @@ public class MyFreeCamsClient {
private int[] ctx; private int[] ctx;
private String ctxenc; private String ctxenc;
private EvictingQueue<String> receivedTextHistory = EvictingQueue.create(100);
private MyFreeCamsClient() { private MyFreeCamsClient() {
moshi = new Moshi.Builder().build(); moshi = new Moshi.Builder().build();
} }
@ -162,6 +166,7 @@ public class MyFreeCamsClient {
@Override @Override
public void onMessage(WebSocket webSocket, String text) { public void onMessage(WebSocket webSocket, String text) {
super.onMessage(webSocket, text); super.onMessage(webSocket, text);
receivedTextHistory.add(text);
msgBuffer.append(text); msgBuffer.append(text);
Message message; Message message;
try { try {
@ -409,18 +414,34 @@ public class MyFreeCamsClient {
// packet size not transmitted completely // packet size not transmitted completely
return null; return null;
} else { } else {
int packetLength = Integer.parseInt(msg.substring(0, 4)); try {
if (packetLength > msg.length() - 4) { int packetLength = Integer.parseInt(msg.substring(0, 4));
// packet not complete if (packetLength > msg.length() - 4) {
// packet not complete
return null;
} else {
msg.delete(0, 4);
int type = parseNextInt(msg);
int sender = parseNextInt(msg);
int receiver = parseNextInt(msg);
int arg1 = parseNextInt(msg);
int arg2 = parseNextInt(msg);
return new Message(type, sender, receiver, arg1, arg2, URLDecoder.decode(msg.toString(), "utf-8"));
}
} catch(Exception e) {
LOG.error("StringBuilder contains invalid data {}", msg.toString(), e);
String logfile = "mfc_messages.log";
try(FileOutputStream fout = new FileOutputStream(logfile)) {
for (String string : receivedTextHistory) {
fout.write(string.getBytes());
fout.write(10);
}
} catch (Exception e1) {
LOG.error("Couldn't write mfc message history to " + logfile);
e1.printStackTrace();
}
msg.setLength(0);
return null; return null;
} else {
msg.delete(0, 4);
int type = parseNextInt(msg);
int sender = parseNextInt(msg);
int receiver = parseNextInt(msg);
int arg1 = parseNextInt(msg);
int arg2 = parseNextInt(msg);
return new Message(type, sender, receiver, arg1, arg2, URLDecoder.decode(msg.toString(), "utf-8"));
} }
} }
} }