build(deps): Electron を 44.2.0 に更新して mp4 再生後の通信断を解消 - #4
Merged
Merged
Conversation
会話内の mp4 を再生すると 10〜30% で止まり、以後サーバーへの全リクエストが
net::ERR_FAILED 200 (OK) で失敗して他の会話も開けなくなる不具合の対策。
原因は protocol.handle('https') が全リクエストを net.fetch に中継する構造で、
<video> の Range リクエスト中断が Electron 43 系の中継を壊すこと
(上流 nextcloud#1401)。Electron 44.2.0 では net.fetch の応答を
JS を経由せず直接中継するようになり(electron/electron#52953)、実機で
Range 要求 38 件・中断 18 件を流しても失敗 0 件を確認した。
上流も nextcloud#1890 で 44.2.0 に更新済み。44 の破壊的変更
(renderer からの clipboard 削除、ログイン項目属性の削除、32bit 廃止)は
本フォークの使用箇所に該当しない。
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YFRUwo6FcYKrJf9htNUouA
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
会話内の mp4 を再生すると 10〜30% で止まり、以後サーバーへの全リクエストが
net::ERR_FAILED 200 (OK)で失敗して他の会話も開けなくなる不具合(Talk Desktop のみ発生)の対策として、Electron を 43 系(実体 43.4.0)から 44.2.0 に更新する。原因
src/app/appProtocol.tsのprotocol.handle('https')が全リクエストをnet.fetchに中継しており、<video>の Range リクエスト中断が Electron 43 系の中継を壊す。上流 nextcloud#1401 と同一。participants/stateの 404 は通信断で Talk セッションが失効した二次症状。対策の根拠
protocol.handleがnet.fetchの応答を JS を経由せず直接中継する(perf: relay untouched net.fetch responses from protocol.handle without pumping them through JS electron/electron#52953、44 系にバックポート)。clipboard削除、ログイン項目属性の削除、32bit 廃止、select-client-certificate)は本フォークの使用箇所に該当なし。検証
CHANNEL=stableで win32 x64 ビルド成功、version= 44.2.0、独自 spreed 同梱を確認。out/make/wix/x64/Nextcloud.Talk-windows-x64.msi、244MB)をインストールして動作確認済み。🤖 Generated with Claude Code
https://claude.ai/code/session_01YFRUwo6FcYKrJf9htNUouA