Skip to content

Commit 287a040

Browse files
committed
ref(logging): add error logging for SampleResult with no response data
Signed-off-by: Johnny Hujol <itudoben@users.noreply.github.com>
1 parent dfef9ee commit 287a040

File tree

1 file changed

+18
-0
lines changed

1 file changed

+18
-0
lines changed

src/main/java/io/github/delirius325/jmeter/backendlistener/elasticsearch/ElasticsearchBackendClient.java

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -272,6 +272,24 @@ public void handleSampleResults(List<SampleResult> results, BackendListenerConte
272272
final String filtersAsString = (this.filters.isEmpty()) ? "no filters" : String.join(", ", this.filters);
273273
for (SampleResult sr : results) {
274274
String respString = sr.getResponseDataAsString();
275+
if (sr.getBytesAsLong() == 0 || respString == null) {
276+
logger.error("SampleResult has no response data. Details: label={}, success={}, responseCode={}, responseMessage={}, " +
277+
"startTime={}, endTime={}, threadName={}, bytes={}, dataType={}, samplerData={}, requestHeaders={}, responseHeaders={}",
278+
sr.getSampleLabel(),
279+
sr.isSuccessful(),
280+
sr.getResponseCode(),
281+
sr.getResponseMessage(),
282+
sr.getStartTime(),
283+
sr.getEndTime(),
284+
sr.getThreadName(),
285+
sr.getBytesAsLong(),
286+
sr.getDataType(),
287+
sr.getSamplerData(),
288+
sr.getRequestHeaders(),
289+
sr.getResponseHeaders());
290+
throw new IllegalStateException("SampleResult has no response data for label: " + sr.getSampleLabel());
291+
}
292+
275293
int endIndex = Math.min(100, respString.length());
276294
respString = respString.substring(0, endIndex);
277295
logger.info("Handle sampler result ({}) {}: {}", endIndex, sr.getSampleLabel(), respString);

0 commit comments

Comments
 (0)