fix: cut too long event messages to max 512 and tests
This commit is contained in:
parent
36364ed607
commit
eba81f5715
2 changed files with 19 additions and 2 deletions
|
@ -103,7 +103,7 @@ public class ClientEvent implements Entity, IndicatorValueHolder {
|
||||||
this.clientTime = (clientTime != null) ? clientTime : 0;
|
this.clientTime = (clientTime != null) ? clientTime : 0;
|
||||||
this.serverTime = (serverTime != null) ? serverTime : Utils.getMillisecondsNow();
|
this.serverTime = (serverTime != null) ? serverTime : Utils.getMillisecondsNow();
|
||||||
this.numValue = numValue;
|
this.numValue = numValue;
|
||||||
this.text = text;
|
this.text = Utils.truncateText(text, 512);
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
|
|
|
@ -520,6 +520,7 @@ public class SEBClientBot {
|
||||||
private final String eventBodyTemplate =
|
private final String eventBodyTemplate =
|
||||||
"{ \"type\": \"%s\", \"timestamp\": %s, \"text\": \"some error " + UUID.randomUUID() + " \" }";
|
"{ \"type\": \"%s\", \"timestamp\": %s, \"text\": \"some error " + UUID.randomUUID() + " \" }";
|
||||||
|
|
||||||
|
private boolean first = true;
|
||||||
private long timestamp = 0;
|
private long timestamp = 0;
|
||||||
private final String eventType;
|
private final String eventType;
|
||||||
|
|
||||||
|
@ -534,7 +535,23 @@ public class SEBClientBot {
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public String getBody() {
|
public String getBody() {
|
||||||
return String.format(this.eventBodyTemplate, this.eventType, this.timestamp);
|
if (this.first) {
|
||||||
|
this.first = false;
|
||||||
|
final String longText =
|
||||||
|
"DEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHT"
|
||||||
|
+ "DEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHT"
|
||||||
|
+ "DEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHT"
|
||||||
|
+ "DEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHT"
|
||||||
|
+ "DEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHT"
|
||||||
|
+ "DEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHT"
|
||||||
|
+ "DEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHT"
|
||||||
|
+ "DEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHTDEDSFGREZHT";
|
||||||
|
final String template =
|
||||||
|
"{ \"type\": \"%s\", \"timestamp\": %s, \"text\": \"some error " + longText + " \" }";
|
||||||
|
return String.format(template, this.eventType, this.timestamp);
|
||||||
|
} else {
|
||||||
|
return String.format(this.eventBodyTemplate, this.eventType, this.timestamp);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
|
|
Loading…
Add table
Reference in a new issue