Skip to content

fix(core): handle streamer error gracefully#2669

Merged
onur-ozkan merged 1 commit intodevfrom
os-signal-hotfix
Oct 28, 2025
Merged

fix(core): handle streamer error gracefully#2669
onur-ozkan merged 1 commit intodevfrom
os-signal-hotfix

Conversation

@onur-ozkan
Copy link
Copy Markdown

Current dev fails on any of the [SIGINT, SIGTERM, SIGQUIT] signals. This fixes that.

Signed-off-by: Onur Özkan <work@onurozkan.dev>
@onur-ozkan onur-ozkan changed the title hotfix: handle streamer error gracefully fix(core): handle streamer error gracefully Oct 28, 2025
@smk762 smk762 self-requested a review October 28, 2025 05:55
@smk762
Copy link
Copy Markdown

smk762 commented Oct 28, 2025

In linux terminal, all of the following successfully exited ./kdf:

  • pkill -9 kdf
  • kill -9 $(pidof kdf)
  • kill $(pidof kdf)
  • Ctl-C
  • the stop RPC method.

LGTM, thanks!

@onur-ozkan onur-ozkan merged commit 7a22854 into dev Oct 28, 2025
21 of 31 checks passed
@onur-ozkan onur-ozkan deleted the os-signal-hotfix branch October 28, 2025 05:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants