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

推荐订阅源

云风的 BLOG
云风的 BLOG
M
MIT News - Artificial intelligence
博客园 - Franky
J
Java Code Geeks
V
Visual Studio Blog
G
Google Developers Blog
罗磊的独立博客
MongoDB | Blog
MongoDB | Blog
奇客Solidot–传递最新科技情报
奇客Solidot–传递最新科技情报
Recent Announcements
Recent Announcements
Last Week in AI
Last Week in AI
Cyber Security Advisories - MS-ISAC
Cyber Security Advisories - MS-ISAC
Stack Overflow Blog
Stack Overflow Blog
博客园 - 司徒正美
The GitHub Blog
The GitHub Blog
腾讯CDC
阮一峰的网络日志
阮一峰的网络日志
V
V2EX
博客园 - 【当耐特】
IT之家
IT之家
I
InfoQ
U
Unit 42
C
Check Point Blog
Martin Fowler
Martin Fowler

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(mattermost): stream guarded api responses · openclaw/...
vincentkoc · 2026-06-19 · via Recent Commits to openclaw:main
11

// Mattermost tests cover client plugin behavior.

22

import { describe, expect, it, vi } from "vitest";

3+4+

const fetchWithSsrFGuardMock = vi.hoisted(() => vi.fn());

5+6+

vi.mock("openclaw/plugin-sdk/ssrf-runtime", async (importOriginal) => {

7+

const actual = await importOriginal<typeof import("openclaw/plugin-sdk/ssrf-runtime")>();

8+

return {

9+

...actual,

10+

fetchWithSsrFGuard: (...args: unknown[]) => fetchWithSsrFGuardMock(...args),

11+

};

12+

});

13+314

import {

415

createMattermostClient,

516

createMattermostPost,

@@ -49,6 +60,55 @@ function parseRequestJson(init: RequestInit | undefined): Record<string, unknown

4960

return parsed as Record<string, unknown>;

5061

}

516263+

function streamingMattermostResponse(body: unknown): {

64+

response: Response;

65+

arrayBuffer: ReturnType<typeof vi.fn>;

66+

} {

67+

const encoded = new TextEncoder().encode(JSON.stringify(body));

68+

const stream = new ReadableStream<Uint8Array>({

69+

start(controller) {

70+

controller.enqueue(encoded);

71+

controller.close();

72+

},

73+

});

74+

const arrayBuffer = vi.fn(async () => {

75+

throw new Error("guarded Mattermost responses must stay streaming");

76+

});

77+

return {

78+

response: {

79+

ok: true,

80+

status: 200,

81+

statusText: "OK",

82+

headers: new Headers({ "content-type": "application/json" }),

83+

body: stream,

84+

arrayBuffer,

85+

} as unknown as Response,

86+

arrayBuffer,

87+

};

88+

}

89+90+

function cancelTrackedResponse(

91+

text: string,

92+

init: ResponseInit,

93+

): {

94+

response: Response;

95+

wasCanceled: () => boolean;

96+

} {

97+

let canceled = false;

98+

const stream = new ReadableStream<Uint8Array>({

99+

start(controller) {

100+

controller.enqueue(new TextEncoder().encode(text));

101+

},

102+

cancel() {

103+

canceled = true;

104+

},

105+

});

106+

return {

107+

response: new Response(stream, init),

108+

wasCanceled: () => canceled,

109+

};

110+

}

111+52112

function createTestClient(response?: { status?: number; body?: unknown; contentType?: string }) {

53113

const { mockFetch, calls } = createMockFetch(response);

54114

const client = createMattermostClient({

@@ -98,6 +158,72 @@ describe("normalizeMattermostBaseUrl", () => {

98158

// ── createMattermostClient ───────────────────────────────────────────

99159100160

describe("createMattermostClient", () => {

161+

it("keeps guarded Mattermost responses streaming until callers consume them", async () => {

162+

const release = vi.fn(async () => {});

163+

const { response, arrayBuffer } = streamingMattermostResponse({ id: "u1" });

164+

fetchWithSsrFGuardMock.mockResolvedValueOnce({ response, release });

165+

const client = createMattermostClient({

166+

baseUrl: "https://chat.example.com",

167+

botToken: "test-token",

168+

});

169+170+

await expect(client.request("/users/me")).resolves.toEqual({ id: "u1" });

171+172+

expect(arrayBuffer).not.toHaveBeenCalled();

173+

expect(release).toHaveBeenCalledTimes(1);

174+

});

175+176+

it("bounds and cancels guarded Mattermost error bodies", async () => {

177+

const release = vi.fn(async () => {});

178+

const tracked = cancelTrackedResponse(`${"upstream unavailable ".repeat(512)}tail`, {

179+

status: 503,

180+

statusText: "Service Unavailable",

181+

headers: { "content-type": "text/plain" },

182+

});

183+

fetchWithSsrFGuardMock.mockResolvedValueOnce({ response: tracked.response, release });

184+

const client = createMattermostClient({

185+

baseUrl: "https://chat.example.com",

186+

botToken: "test-token",

187+

});

188+189+

let caught: Error | undefined;

190+

try {

191+

await client.request("/users/me");

192+

} catch (error) {

193+

caught = error as Error;

194+

}

195+196+

expect(caught?.message).toContain("Mattermost API 503 Service Unavailable");

197+

expect(caught?.message).toContain("upstream unavailable");

198+

expect(caught?.message).not.toContain("tail");

199+

expect(caught?.message.length).toBeLessThan(8_300);

200+

expect(tracked.wasCanceled()).toBe(true);

201+

expect(release).toHaveBeenCalledTimes(1);

202+

});

203+204+

it("releases guarded Mattermost responses when upstream body reads fail", async () => {

205+

const release = vi.fn(async () => {});

206+

const stream = new ReadableStream<Uint8Array>({

207+

pull() {

208+

throw new Error("upstream body failed");

209+

},

210+

});

211+

fetchWithSsrFGuardMock.mockResolvedValueOnce({

212+

response: new Response(stream, {

213+

status: 200,

214+

headers: { "content-type": "application/json" },

215+

}),

216+

release,

217+

});

218+

const client = createMattermostClient({

219+

baseUrl: "https://chat.example.com",

220+

botToken: "test-token",

221+

});

222+223+

await expect(client.request("/users/me")).rejects.toThrow("upstream body failed");

224+

expect(release).toHaveBeenCalledTimes(1);

225+

});

226+101227

it("creates a client with normalized baseUrl", () => {

102228

const { mockFetch } = createMockFetch();

103229

const client = createMattermostClient({