Antalya 25.8 - disable crash report in more places#1387
Conversation
|
Small changes only, running regression tests is not necessary. |
QA Verification – Crash report behaviorI verified the crash-report behavior by forcibly crashing the server using Before PRVersion: Server log evidence: The server explicitly enters the crash-report upload path and attempts to send data to Network evidence (
This confirms that the crash report is actually uploaded before the PR. After PRVersion: Server log evidence: Stack trace clearly shows the send path is still executed. Network evidence:
This confirms that the crash report is not sent after the PR (expected behavior), but the server still attempts to send it and fails due to an invalid/empty URL, resulting in a runtime exception. Conclusion
Ideally, when crash reporting is disabled, the send path should be fully skipped (no send attempt and no exception), to avoid misleading errors and unnecessary log noise during crashes. Issue Opened: #1437 |
Changelog category (leave one):
Some embedded config files did not have crash reporting disabled.
CI/CD Options
Exclude tests:
Regression jobs to run: