Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(windows): improve process termination and port cleanup #1151

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

onyedikachi-david
Copy link
Contributor

@onyedikachi-david onyedikachi-david commented Jan 15, 2025

description

Implements robust process management for Windows to fix issues with processes not being properly terminated and ports remaining in use. This PR introduces a new ProcessManager that uses Windows Job Objects to ensure proper cleanup of processes and port releases.

Fixes: #1149

Copy link

vercel bot commented Jan 15, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
screenpipe ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 16, 2025 1:07am

@louis030195
Copy link
Collaborator

maybe can you share screen recording to show it works in different conditions?

biggest issue is when updating windows app and then have to reboot because of this issue sometimes ...

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.

[bounty] solve the windows process not dying issue
2 participants