Skip to content

Commit 255bcbf

Browse files
committed
fix(network): lower slp parse failure logging
1 parent 4651797 commit 255bcbf

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

MCServerLauncher.Common/Network/SlpClient.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -185,7 +185,7 @@ public async Task<bool> HandShakeAsync(string host, ushort port, CancellationTok
185185
}
186186
catch (Exception e)
187187
{
188-
Log.Error("[SlpClient] Failed to parse server ping payload: {0}", e);
188+
Log.Debug(e, "[SlpClient] Failed to parse server ping payload");
189189
return null;
190190
}
191191
}
@@ -218,7 +218,7 @@ public async Task<bool> HandShakeAsync(string host, ushort port, CancellationTok
218218
}
219219
catch (Exception e)
220220
{
221-
Log.Error("[SlpClient] Failed to parse server ping payload: {0}", e);
221+
Log.Debug(e, "[SlpClient] Failed to parse server ping payload");
222222
return null;
223223
}
224224
}

docs/superpowers/plans/2026-06-19-slp-report-failure-tolerance.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -70,3 +70,4 @@ Expected: commands exit 0.
7070
- Added regression coverage for invalid Minecraft SLP status responses closing before latency ping.
7171
- Updated modern SLP probing to stop after invalid status payloads.
7272
- Hardened Minecraft instance report enrichment so player-list probe failures do not fail `GetAllReports`.
73+
- Downgraded invalid SLP payload parse logs from error to debug because report-time Minecraft ping is best-effort.

0 commit comments

Comments
 (0)