Skip to content

fix: only listen to SIGINT and SIGTERM#236

Merged
joe0BAB merged 1 commit into
mainfrom
fix/notify-context
Aug 20, 2025
Merged

fix: only listen to SIGINT and SIGTERM#236
joe0BAB merged 1 commit into
mainfrom
fix/notify-context

Conversation

@joe0BAB

@joe0BAB joe0BAB commented Aug 20, 2025

Copy link
Copy Markdown
Collaborator

Missed this before and seems like without this the daemon is a bit too sensitive / listens to any sigterm signal even if ctrl+c is pressed in another terminal for another process.

@joe0BAB joe0BAB self-assigned this Aug 20, 2025
@joe0BAB joe0BAB requested review from Benehiko and wmluke August 20, 2025 12:33
@joe0BAB joe0BAB added this to the docker-mysecret milestone Aug 20, 2025
@joe0BAB joe0BAB marked this pull request as ready for review August 20, 2025 12:36
@joe0BAB joe0BAB merged commit 2a14857 into main Aug 20, 2025
15 checks passed
@joe0BAB joe0BAB deleted the fix/notify-context branch August 20, 2025 12:58
@joe0BAB joe0BAB linked an issue Aug 25, 2025 that may be closed by this pull request
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.

Disconnecting an external plugin crashes the engine

2 participants