fix(gateway): skip stale-socket restarts for Telegram polling#38405
Merged
Takhoffman merged 4 commits intoopenclaw:mainfrom Mar 7, 2026
Merged
fix(gateway): skip stale-socket restarts for Telegram polling#38405Takhoffman merged 4 commits intoopenclaw:mainfrom
Takhoffman merged 4 commits intoopenclaw:mainfrom
Conversation
Contributor
Greptile SummaryThis PR fixes Telegram's health-monitor triggering a spurious stale-socket restart every ~35 minutes by skipping the stale-socket check when the channel is Key observations:
Confidence Score: 3/5
Last reviewed commit: 65dc78f |
ql-wade
pushed a commit
to ql-wade/openclaw
that referenced
this pull request
Mar 7, 2026
- Import and use ChannelId type for channelId field (type safety) - Fix guard block indentation for Telegram check - Move policy creation after null guard in readiness.ts
598c532 to
61aa11e
Compare
Contributor
|
PR #38405 - fix(gateway): skip stale-socket restarts for Telegram polling (#38405) Merged after verification.
|
6 tasks
mrosmarin
added a commit
to mrosmarin/openclaw
that referenced
this pull request
Mar 7, 2026
* main: (45 commits) chore: update dependencies except carbon test(memory): make mcporter EINVAL retry test deterministic refactor(extensions): reuse shared helper primitives refactor(core): extract shared dedup helpers fix(ci): re-enable detect-secrets on main docs: reorder 2026.3.7 changelog highlights chore: bump version to 2026.3.7 fix(android): align run command with app id docs: add changelog entry for Android package rename (openclaw#38712) fix(android): rename app package to ai.openclaw.app fix(gateway): stop stale-socket restarts before first event (openclaw#38643) fix(gateway): skip stale-socket restarts for Telegram polling (openclaw#38405) fix(gateway): invalidate bootstrap cache on session rollover (openclaw#38535) docs: update changelog for reply media delivery (openclaw#38572) fix: contain final reply media normalization failures fix: contain block reply media failures fix: normalize reply media paths Fix owner-only auth and overlapping skill env regressions (openclaw#38548) fix(feishu): disable block streaming to prevent silent reply drops (openclaw#38422) fix: suppress ACP NO_REPLY fragments in console output (openclaw#38436) ...
mcaxtr
pushed a commit
to mcaxtr/openclaw
that referenced
this pull request
Mar 7, 2026
…aw#38405) Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: ql-wade <[email protected]>
joshavant
pushed a commit
that referenced
this pull request
Mar 7, 2026
Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: ql-wade <[email protected]>
vincentkoc
pushed a commit
to BryanTegomoh/openclaw-fork
that referenced
this pull request
Mar 8, 2026
…aw#38405) Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: ql-wade <[email protected]>
Saitop
pushed a commit
to NomiciAI/openclaw
that referenced
this pull request
Mar 8, 2026
…aw#38405) Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: ql-wade <[email protected]>
jenawant
pushed a commit
to jenawant/openclaw
that referenced
this pull request
Mar 10, 2026
…aw#38405) Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: ql-wade <[email protected]>
dhoman
pushed a commit
to dhoman/chrono-claw
that referenced
this pull request
Mar 11, 2026
…aw#38405) Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: ql-wade <[email protected]>
senw-developers
pushed a commit
to senw-developers/va-openclaw
that referenced
this pull request
Mar 17, 2026
…aw#38405) Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: ql-wade <[email protected]>
V-Gutierrez
pushed a commit
to V-Gutierrez/openclaw-vendor
that referenced
this pull request
Mar 17, 2026
…aw#38405) Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: ql-wade <[email protected]>
alexey-pelykh
pushed a commit
to remoteclaw/remoteclaw
that referenced
this pull request
Mar 20, 2026
…aw#38405) Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: ql-wade <[email protected]> (cherry picked from commit a5c07fa)
alexey-pelykh
pushed a commit
to remoteclaw/remoteclaw
that referenced
this pull request
Mar 20, 2026
…aw#38405) Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: ql-wade <[email protected]> (cherry picked from commit a5c07fa)
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
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.
自动修复 Issue #38395
修复内容
为 Telegram long-polling 模式跳过 stale-socket 检查。
根本原因
Telegram 使用 long-polling 架构,每个请求都作为心跳。stale-socket 检查是为 WebSocket 设计的,用于检测半死连接(连接显示活跃但不发送事件),这不适用于 Telegram 的 polling 架构。
修复方案
测试
Closes #38395