惯性聚合 高效追踪和阅读你感兴趣的博客、新闻、科技资讯
阅读原文 在惯性聚合中打开

推荐订阅源

Hugging Face - Blog
Hugging Face - Blog
Vercel News
Vercel News
C
Check Point Blog
G
Google Developers Blog
博客园 - 司徒正美
量子位
Engineering at Meta
Engineering at Meta
S
SegmentFault 最新的问题
Google DeepMind News
Google DeepMind News
F
Fortinet All Blogs
A
About on SuperTechFans
美团技术团队
D
DataBreaches.Net
Stack Overflow Blog
Stack Overflow Blog
Jina AI
Jina AI
Y
Y Combinator Blog
奇客Solidot–传递最新科技情报
奇客Solidot–传递最新科技情报
Apple Machine Learning Research
Apple Machine Learning Research
J
Java Code Geeks
MongoDB | Blog
MongoDB | Blog
人人都是产品经理
人人都是产品经理
H
Hackread – Cybersecurity News, Data Breaches, AI and More
The Cloudflare Blog
U
Unit 42

Recent Commits to openclaw:main

test: merge chat side-result checks · openclaw/openclaw@ddd2c2a test: merge cron history checks · openclaw/openclaw@f7eb746 test: merge responsive navigation shell checks · openclaw/openclaw@c2e4b47 docs(changelog): add codex oauth fixes · openclaw/openclaw@628e6cd test: merge navigation routing cases · openclaw/openclaw@5d8cecb Tests: mock channel registry bundled fallback · openclaw/openclaw@2b08233 Secrets: avoid broad web search discovery for single plugin config · openclaw/openclaw@a464f59 test: merge config view browser checks · openclaw/openclaw@20cf511 fix(status): align oauth health with runtime · openclaw/openclaw@eed7116 feat: add macOS screen snapshots for monitor preview (#67954) thanks … · openclaw/openclaw@f377db1 fix: report shared auth scopes in hello-ok (#67810) thanks @BunsDev · openclaw/openclaw@0b6c39b Auto-reply: avoid eager bundled route fallback · openclaw/openclaw@3ea1bf4 Tests: narrow session binding contract setup · openclaw/openclaw@54e4e16 fix(macOS): enable undo/redo in webchat composer text input (#34962) · openclaw/openclaw@00951dc Tests: speed up channel setup promotion · openclaw/openclaw@82b529a Docs: refresh agent instructions · openclaw/openclaw@5775fe2 fix(auth): serialize OAuth refresh across agents to fix #26322 (#67876) · openclaw/openclaw@8e79080 test: allow ollama public surface boundary test · openclaw/openclaw@7d4f1a6 Docs: add test performance guardrails · openclaw/openclaw@89706d3 Tests: restore context-engine usage proof · openclaw/openclaw@e4c4f95 Tests: slim context engine runtime coverage · openclaw/openclaw@74c198f ci: retry failed custom checkouts · openclaw/openclaw@0ee5baf test: trim duplicate provider auth onboarding cases · openclaw/openclaw@1ffc02e matrix: fix sessions_spawn --thread subagent session spawning (#67643) · openclaw/openclaw@1ce2596 test: reduce auth choice fixture churn · openclaw/openclaw@857b9cd test: mock health status config boundaries · openclaw/openclaw@9d5ab4a test: mock onboard config io boundary · openclaw/openclaw@299694d test: mock legacy state plugin boundaries · openclaw/openclaw@2713089 test: mock channel install boundaries · openclaw/openclaw@b945248 test: mock doctor preview channel boundaries · openclaw/openclaw@b1a3ad4
fix(telegram): retry 421 misdirected request responses · ...
MarsDoge · 2026-05-18 · via Recent Commits to openclaw:main
Original file line numberDiff line numberDiff line change

@@ -246,6 +246,44 @@ describe("createTelegramBot fetch abort", () => {

246246

vi.useRealTimers();

247247

});

248248
249+

it("retries Telegram 421 responses after forcing transport fallback", async () => {

250+

const forceFallback = vi.fn(() => true);

251+

const fetchSpy = vi

252+

.fn()

253+

.mockResolvedValueOnce(new Response("Misdirected Request", { status: 421 }))

254+

.mockResolvedValueOnce(new Response("{}", { status: 200 }));

255+

const { clientFetch } = createWrappedTelegramClientFetchWithTransport({

256+

fetch: fetchSpy as typeof fetch,

257+

forceFallback,

258+

});

259+
260+

const result = await clientFetch("https://api.telegram.org/bot123456:ABC/sendMessage");

261+
262+

expect(result).toBeInstanceOf(Response);

263+

expect((result as Response).status).toBe(200);

264+

expect(forceFallback).toHaveBeenCalledWith("misdirected-request");

265+

expect(fetchSpy).toHaveBeenCalledTimes(2);

266+

});

267+
268+

it("retries Telegram 421 fetch errors after forcing transport fallback", async () => {

269+

const forceFallback = vi.fn(() => true);

270+

const fetchSpy = vi

271+

.fn()

272+

.mockRejectedValueOnce(Object.assign(new Error("421 Misdirected Request"), { status: 421 }))

273+

.mockResolvedValueOnce(new Response("{}", { status: 200 }));

274+

const { clientFetch } = createWrappedTelegramClientFetchWithTransport({

275+

fetch: fetchSpy as typeof fetch,

276+

forceFallback,

277+

});

278+
279+

const result = await clientFetch("https://api.telegram.org/bot123456:ABC/sendMessage");

280+
281+

expect(result).toBeInstanceOf(Response);

282+

expect((result as Response).status).toBe(200);

283+

expect(forceFallback).toHaveBeenCalledWith("misdirected-request");

284+

expect(fetchSpy).toHaveBeenCalledTimes(2);

285+

});

286+
249287

it("preserves the original fetch error when tagging cannot attach metadata", async () => {

250288

const frozenError = Object.freeze(

251289

Object.assign(new TypeError("fetch failed"), {

Original file line numberDiff line numberDiff line change

@@ -1,7 +1,7 @@

11

import type { ApiClientOptions } from "grammy";

22

import { normalizeOptionalLowercaseString } from "openclaw/plugin-sdk/string-coerce-runtime";

33

import type { TelegramTransport } from "./fetch.js";

4-

import { tagTelegramNetworkError } from "./network-errors.js";

4+

import { isTelegramMisdirectedRequestError, tagTelegramNetworkError } from "./network-errors.js";

55

import { resolveTelegramRequestTimeoutMs } from "./request-timeouts.js";

66
77

type TelegramFetchInput = Parameters<NonNullable<ApiClientOptions["fetch"]>>[0];

@@ -135,6 +135,11 @@ export function createTelegramClientFetch(params: {

135135

: undefined;

136136

const requestSignal = isTelegramAbortSignalLike(init?.signal) ? init.signal : undefined;

137137
138+

const canForceTransportFallback = (reason: string) =>

139+

!shutdownSignal?.aborted &&

140+

!requestSignal?.aborted &&

141+

params.transport?.forceFallback?.(reason) === true;

142+
138143

const runFetch = async () => {

139144

const controller = new AbortController();

140145

const abortWith = (signal: Pick<TelegramAbortSignalLike, "reason">) =>

@@ -195,14 +200,22 @@ export function createTelegramClientFetch(params: {

195200

};

196201
197202

try {

198-

return await runFetch();

203+

const response = await runFetch();

204+

if (response.status === 421 && canForceTransportFallback("misdirected-request")) {

205+

return await runFetch();

206+

}

207+

return response;

199208

} catch (err) {

200209

if (

201210

requestTimeoutMs &&

202211

shouldRetryTimedOutTelegramControlRequest(method) &&

203-

!shutdownSignal?.aborted &&

204-

!requestSignal?.aborted &&

205-

params.transport?.forceFallback?.("request-timeout")

212+

canForceTransportFallback("request-timeout")

213+

) {

214+

return await runFetch();

215+

}

216+

if (

217+

isTelegramMisdirectedRequestError(err) &&

218+

canForceTransportFallback("misdirected-request")

206219

) {

207220

return await runFetch();

208221

}

Original file line numberDiff line numberDiff line change

@@ -253,6 +253,36 @@ describe("isSafeToRetrySendError", () => {

253253

);

254254

expect(isSafeToRetrySendError(wrapped)).toBe(false);

255255

});

256+
257+

it.each([

258+

["status", Object.assign(new Error("Misdirected Request"), { status: 421 })],

259+

["statusCode", Object.assign(new Error("Misdirected Request"), { statusCode: "421" })],

260+

["error_code", errorWithTelegramCode("Misdirected Request", 421)],

261+

["message", new Error("421 Misdirected Request")],

262+

[

263+

"nested cause",

264+

Object.assign(new Error("Network request for 'sendMessage' failed!"), {

265+

cause: Object.assign(new Error("Misdirected Request"), { status: 421 }),

266+

}),

267+

],

268+

[

269+

"grammY HttpError",

270+

new MockHttpError(

271+

"Network request for 'sendMessage' failed!",

272+

Object.assign(new Error("Misdirected Request"), { status: 421 }),

273+

),

274+

],

275+

])("treats Telegram 421 Misdirected Request as safe to retry via %s", (_name, err) => {

276+

expect(isSafeToRetrySendError(err)).toBe(true);

277+

});

278+
279+

it("does not parse malformed status strings as Telegram 421", () => {

280+

expect(

281+

isSafeToRetrySendError(

282+

Object.assign(new Error("Misdirected Request"), { statusCode: "421abc" }),

283+

),

284+

).toBe(false);

285+

});

256286

});

257287
258288

describe("isTelegramServerError", () => {

Original file line numberDiff line numberDiff line change

@@ -103,6 +103,40 @@ function getErrorCode(err: unknown): string | undefined {

103103

return undefined;

104104

}

105105
106+

function getNumericHttpStatus(err: unknown): number | undefined {

107+

if (!err || typeof err !== "object") {

108+

return undefined;

109+

}

110+

const candidate = err as { error_code?: unknown; status?: unknown; statusCode?: unknown };

111+

for (const value of [candidate.error_code, candidate.status, candidate.statusCode]) {

112+

if (typeof value === "number" && Number.isFinite(value)) {

113+

return value;

114+

}

115+

if (typeof value === "string") {

116+

const trimmed = value.trim();

117+

if (/^\d+$/.test(trimmed)) {

118+

return Number.parseInt(trimmed, 10);

119+

}

120+

}

121+

}

122+

return undefined;

123+

}

124+
125+

export function isTelegramMisdirectedRequestError(err: unknown): boolean {

126+

for (const candidate of collectTelegramErrorCandidates(err)) {

127+

const code = normalizeCode(getErrorCode(candidate));

128+

if (code === "421" || getNumericHttpStatus(candidate) === 421) {

129+

return true;

130+

}

131+
132+

const message = normalizeLowercaseStringOrEmpty(formatErrorMessage(candidate));

133+

if (/\b421\b/.test(message) && message.includes("misdirected request")) {

134+

return true;

135+

}

136+

}

137+

return false;

138+

}

139+
106140

export type TelegramNetworkErrorContext = "polling" | "send" | "webhook" | "unknown";

107141

export type TelegramNetworkErrorOrigin = {

108142

method?: string | null;

@@ -162,6 +196,9 @@ export function isSafeToRetrySendError(err: unknown): boolean {

162196

if (!err) {

163197

return false;

164198

}

199+

if (isTelegramMisdirectedRequestError(err)) {

200+

return true;

201+

}

165202

for (const candidate of collectTelegramErrorCandidates(err)) {

166203

const code = normalizeCode(getErrorCode(candidate));

167204

if (code && PRE_CONNECT_ERROR_CODES.has(code)) {

Original file line numberDiff line numberDiff line change

@@ -160,6 +160,24 @@ describe("createChannelApiRetryRunner", () => {

160160

});

161161

});

162162
163+

describe("default retry behavior", () => {

164+

it("retries misdirected request errors from Telegram edge nodes", async () => {

165+

await runRetryCase({

166+

runnerOptions: { retry: ZERO_DELAY_RETRY },

167+

fnSteps: [

168+

{

169+

type: "reject" as const,

170+

value: Object.assign(new Error("421 Misdirected Request"), {

171+

status: 421,

172+

}),

173+

},

174+

],

175+

expectedCalls: 3,

176+

expectedError: "421 Misdirected Request",

177+

});

178+

});

179+

});

180+
163181

it("honors nested retry_after hints before retrying", async () => {

164182

vi.useFakeTimers();

165183
Original file line numberDiff line numberDiff line change

@@ -11,7 +11,8 @@ export const CHANNEL_API_RETRY_DEFAULTS = {

1111

jitter: 0.1,

1212

};

1313
14-

const CHANNEL_API_RETRY_RE = /429|timeout|connect|reset|closed|unavailable|temporarily/i;

14+

const CHANNEL_API_RETRY_RE =

15+

/429|421|timeout|connect|reset|closed|unavailable|temporarily|misdirected request/i;

1516

const log = createSubsystemLogger("retry-policy");

1617
1718

function resolveChannelApiShouldRetry(params: {