Skip to content

OnlineServices: Implement graceful shutdown of online services - #61

Merged
x64-dev merged 1 commit into
mainfrom
seer/online-services-shutdown
Oct 10, 2025
Merged

OnlineServices: Implement graceful shutdown of online services#61
x64-dev merged 1 commit into
mainfrom
seer/online-services-shutdown

Conversation

@seer-by-sentry

@seer-by-sentry seer-by-sentry Bot commented Oct 9, 2025

Copy link
Copy Markdown

Fixes CLIENT-68. The issue was that: Online services manager deletion occurs before background worker threads terminate, causing a write access violation on freed memory.

  • Added shutdown logic to OnlineServicesManager, HTTPManager, and WebSocket to ensure proper cleanup and prevent crashes during exit.
  • Introduced atomic boolean flags to signal shutdown state in HTTPManager and WebSocket.
  • Implemented thread joining for screenshot threads in OnlineServicesManager to prevent access to destroyed objects.
  • Added logging to track shutdown progress and identify potential issues.

This fix was generated by Seer in Sentry, triggered by x64-dev. 👁️ Run ID: 1852757

Not quite right? Click here to continue debugging with Seer.

@x64-dev
x64-dev marked this pull request as ready for review October 10, 2025 21:22
@x64-dev
x64-dev merged commit 617baae into main Oct 10, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant