Close stray process when stdio pipe is closed or possibly when parent process is killed (daemon flag?)
まだ誰も着手していません。
評価
調査の方向性
まず issue #655 と PR #663 を確認し、次に -stdio モードと提案されている -daemon フラグに対応する PSES のエントリーポイントを追跡します。親プロセスが消失した場合、または stdio パイプが閉じた場合に、どのシャットダウン動作が意図されているのかを判断してください。対象となるシナリオで残存する PSES プロセスがなくなり、daemon の使用を壊さなければ完了です。
索引モデルが issue の本文から書いたものです。
説明
This is continuation of the effort from:
- Issue https://github.com/PowerShell/PowerShellEditorServices/issues/655 PSES needs to shutdown itself when it's supposed to
- PR https://github.com/PowerShell/PowerShellEditorServices/pull/663 Close stray processes on exit
#655 was closed by @TylerLeonhardt as it was satisfied by PR #663. However I still believe we can easily get into the situation when LSP text editor that launches PSES gets killed and orphans the PSES process, that keeps running without a purpose. Actually this is the case with Sublime Text and LSP plugin, where other language LSP providers shutdown themselves when the Sublime Text quits without a need to use exit or any other means.
Therefore I propose 2 options for consideration (alternatives):
-
Introduce new
-daemonflag, that unless specified, makes PSES to shutdown itself when the parent process disappears. This will make PSES behave like most of the command line tools, i.e. shutdown itself when the shell or launching process quits.The only purpose for PSES to run despite of parent process being alive or not is when it is it intended to be ran as a daemon, and I believe this intent should not be default (as it is now).
-
Shutdown itself when running as
-stdioand when pipe is closed, as I was commenting in other ticket, there is no point running when pipe is closes, because there is nobody that will be able to issueexitand any other command anyways.
Thank you in advance for considering my suggestion. If you prefer I can try to get this behavior adapter myself and send the PR, but you (maintainers) may come to more elegant solution much quicker than me, therefore I post this issue first.
- 主要言語
- C#
- スター
- 767
- フォーク
- 266
- 平均マージ
- 3日 16時間
- マージ済み PR（30日）
- 1
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
PowerShell/PowerShellEditorServices のほかの issue
-
[Bug] None of the string overloads of `EditorExtensionServiceProvider.GetService()` work on .NET. オープン
PowerShell/PowerShellEditorServices#2331 · リアクション 1 件 · 担当者 2 名 ·
-
Area-Language Server Issue-Enhancement
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
PowerShell/PowerShellEditorServices#2315 · コメント 1 件 ·
-
Area-Debugging Issue-Enhancement
PowerShell/PowerShellEditorServices#2294 · リアクション 1 件 · 担当者 2 名 ·
-
Issue-Bug Needs: Triage
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
PowerShell/PowerShellEditorServices#2293 · コメント 1 件 ·
-
PowerShell/PowerShellEditorServices#2282 · コメント 1 件 · リアクション 1 件 · 担当者 2 名 ·
PowerShell/PowerShellEditorServices の issue をすべて見る
似ている issue
-
Documentation
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
cake-build/cake#5024 ·
-
Frontend status/draft TechnicalDebt
難易度 2/5 1〜2日 初心者へのやさしさ 75/100
Altinn/altinn-auth#4143 ·
-
.NET Flaky Test Testing Tests
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
getsentry/sentry-dotnet#5617 · コメント 1 件 ·
-
:watch: Not Triaged dotnet-fsharp/svc
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
-
Client customer-reported needs-team-attention question Service Attention WebPubSub
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
Azure/azure-sdk-for-net#63292 · コメント 3 件 · リアクション 1 件 ·