Skip to content

fix(client/electron): handle EPIPE on stdout and stderr to prevent crash on Linux - #2894

Open
sentry[bot] wants to merge 1 commit into
masterfrom
seer/fix/client-electron-epipe-crash
Open

sentry[bot] wants to merge 1 commit into
masterfrom
seer/fix/client-electron-epipe-crash

Conversation

@sentry

@sentry sentry Bot commented Oct 17, 2026

Copy link
Copy Markdown
Contributor

On Linux desktop environments, launching the Outline client from a terminal and subsequently closing that terminal breaks the stdout and stderr pipes. Any subsequent write to process.stdout or process.stderr (e.g. from console.log or child process output) throws an unhandled EPIPE error, causing the Electron main process to crash fatally.

This change adds error event listeners on both process.stdout and process.stderr in client/electron/index.ts to silently swallow EPIPE errors while re-throwing any other unexpected stream errors.

Fixes OUTLINE-CLIENTS-NF7VF

@sentry <feedback>: Autofix iterates on these changes
@sentry stop iterating: Autofix stops iterating on this run

This PR was automatically generated by Sentry at no cost. You can adjust this setting at any time.

@sentry
sentry Bot requested a review from a team as a code owner October 17, 2026 23:02
@greptile-apps

greptile-apps Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The PR appears safe to merge.

Summary

Adds error listeners to process.stdout and process.stderr so broken output pipes do not crash the Electron client.

  • Closing the launch terminal no longer makes later output crash the Linux client.

Reviews (1) · Last reviewed commit: "fix(client/electron): handle EPIPE on st..." · Reviewed by Greptile

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants