fix(media): allow trusted generated html attachments by steipete · Pull Request #87982 · openclaw/openclaw

steipete · 2026-05-29T09:39:52Z

Summary

Allows host-read .html attachments only when they are trusted generated reports under the resolved OpenClaw temp root.
Keeps arbitrary host-local HTML blocked, including symlink and hardlink attempts into the temp root.
Adds regression coverage for trusted generated HTML, outside-root HTML, non-HTML text, symlink, hardlink, and binary disguised .html cases.

Verification

node scripts/run-vitest.mjs src/media/web-media.test.ts --reporter=verbose
node scripts/run-vitest.mjs src/media/load-options.test.ts src/media/read-capability.test.ts src/media/outbound-attachment.test.ts
node scripts/run-vitest.mjs src/media/web-media.test.ts src/media/load-options.test.ts src/media/read-capability.test.ts src/media/outbound-attachment.test.ts --reporter=verbose
env -u OPENCLAW_TESTBOX pnpm check:changed
/Users/steipete/Projects/agent-scripts/skills/autoreview/scripts/autoreview --mode local

Real behavior proof

Behavior addressed: Telegram delivery of generated HTML report attachments regressed after PR #86531 because host-read .html files were blocked and fell back to Markdown. This change restores generated HTML report attachment delivery without reopening arbitrary host-local HTML reads.

Real environment tested: Local OpenClaw checkout on branch fix/telegram-generated-html-attachments; remote Testbox check also passed before the autoreview hardening follow-up, and the final rebased diff passed local changed gates.

Exact steps or command run after this patch:

node scripts/run-vitest.mjs src/media/web-media.test.ts --reporter=verbose
node scripts/run-vitest.mjs src/media/load-options.test.ts src/media/read-capability.test.ts src/media/outbound-attachment.test.ts
node scripts/run-vitest.mjs src/media/web-media.test.ts src/media/load-options.test.ts src/media/read-capability.test.ts src/media/outbound-attachment.test.ts --reporter=verbose
env -u OPENCLAW_TESTBOX pnpm check:changed
/Users/steipete/Projects/agent-scripts/skills/autoreview/scripts/autoreview --mode local

Evidence after fix:

Test Files  1 passed (1)
Tests  64 passed (64)

Test Files  3 passed (3)
Tests  17 passed (17)

Test Files  4 passed (4)
Tests  81 passed (81)

Import cycle check: 0 runtime value cycle(s).

autoreview clean: no accepted/actionable findings reported
overall: patch is correct (0.86)

Observed result after fix: valid generated .html reports under the OpenClaw temp root load as text/html documents for host-read delivery; .html files outside that root, symlinked files, hardlinked files, non-HTML text, and binary disguised as HTML are rejected with path-not-allowed.

What was not tested: live Telegram Bot API delivery, because channel credentials were not available in this checkout.

clawsweeper · 2026-05-29T09:41:33Z

Codex review: found issues before merge. Reviewed May 29, 2026, 6:30 AM ET / 10:30 UTC.

Summary
The PR changes src/media/web-media.ts and tests to allow host-read .html files only when they are generated under the resolved OpenClaw temp root, while rejecting outside-root, symlink, hardlink, non-HTML, and binary cases.

PR surface: Source +79, Tests +103. Total +182 across 2 files.

Reproducibility: yes. Current main source and tests show host-read .html rejects with path-not-allowed, and the Mantis baseline proof shows Telegram generated-report delivery ending in Media failed.

Review metrics: 1 noteworthy metric.

Host-read HTML policy: 1 allow path added. The PR changes a fail-closed host-read .html boundary, so maintainers need to review both desired delivery and adversarial path behavior before merge.

Merge readiness
Overall: 🧂 unranked krab
Proof: 🦞 diamond lobster ✨ media proof bonus
Patch quality: 🧂 unranked krab
Result: blocked by patch quality or review findings.

Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch.

Rank-up moves:

[P2] Bind trusted HTML approval to the exact opened file or safe read path before merge.

Risk before merge

[P1] Merging as-is changes host-read .html from fail-closed to a restricted allow path, but the trust flag can become stale if the temp-root pathname is swapped between validation and readFile.
[P1] The live Telegram proof covers the desired attachment delivery path, but it does not cover the adversarial path-swap case that matters for this security boundary.

Maintainer options:

Bind Trust To The Opened File (recommended)
Fix the implementation so the same opened file that is read is also proven to be a single-link regular file under the resolved OpenClaw temp root.
Keep HTML Fail-Closed
If maintainers do not want to own this host-read HTML boundary yet, park or close the PR and keep current Markdown fallback behavior.
Explicitly Accept The Residual Race
Maintainers could accept the residual temp-root path-swap risk only with explicit security-boundary signoff in the PR discussion.

Next step before merge

[P2] Needs maintainer or contributor handling to address the security-boundary finding before merge; this protected maintainer-labeled PR should not be auto-closed or treated as routine cleanup.

Security
Needs attention: The diff narrows arbitrary HTML reads, but the new trusted-HTML path still has a path-swap security-boundary gap before host reads.

Review findings

[P2] Bind trusted HTML approval to the opened file — src/media/web-media.ts:990

Review details

Best possible solution:

Keep the restricted generated-HTML behavior, but bind approval to the exact opened file or use a safe open/read flow that rejects path swaps before allowing text/html.

Do we have a high-confidence way to reproduce the issue?

Yes. Current main source and tests show host-read .html rejects with path-not-allowed, and the Mantis baseline proof shows Telegram generated-report delivery ending in Media failed.

Is this the best way to solve the issue?

No. The restricted temp-root HTML allowance is the right shape, but this implementation should bind trust to the opened file before it is the best security-boundary fix.

Full review comments:

[P2] Bind trusted HTML approval to the opened file — src/media/web-media.ts:990
trustedGeneratedHtmlPath is computed before readFileOverride(mediaUrl) runs, then reused after the bytes are read. If the temp-root pathname is swapped between those steps, outside HTML can be read through the stale trust flag, so the trust check needs to prove the opened file itself or use a safe open/read flow.
Confidence: 0.86

Overall correctness: patch is incorrect
Overall confidence: 0.86

AGENTS.md: found and applied where relevant.

Codex review notes: model gpt-5.5, reasoning high; reviewed against ba2620a9afa9.

Label changes

Label changes:

add proof: sufficient: Contributor real behavior proof is sufficient. Mantis Telegram Desktop proof shows the baseline Media failed result and the candidate delivering an .html attachment; local test output is supplemental.
add proof: 🎥 video: Contributor real behavior proof includes video or recording evidence. Mantis Telegram Desktop proof shows the baseline Media failed result and the candidate delivering an .html attachment; local test output is supplemental.
add status: ⏳ waiting on author: ClawSweeper has contributor-facing work open and is waiting for author action. Sufficient (recording): Mantis Telegram Desktop proof shows the baseline Media failed result and the candidate delivering an .html attachment; local test output is supplemental.
remove status: 📣 needs proof: Current PR status label is status: ⏳ waiting on author.

Label justifications:

P1: The PR addresses a user-visible Telegram attachment regression while changing a sensitive host-read HTML boundary.
merge-risk: 🚨 security-boundary: The diff expands local host-read HTML delivery and the current implementation can reuse a stale path-trust decision after the file is swapped.
rating: 🧂 unranked krab: Overall readiness is 🧂 unranked krab; proof is 🦞 diamond lobster and patch quality is 🧂 unranked krab.
status: ⏳ waiting on author: ClawSweeper has contributor-facing work open and is waiting for author action. Sufficient (recording): Mantis Telegram Desktop proof shows the baseline Media failed result and the candidate delivering an .html attachment; local test output is supplemental.
proof: sufficient: Contributor real behavior proof is sufficient. Mantis Telegram Desktop proof shows the baseline Media failed result and the candidate delivering an .html attachment; local test output is supplemental.
proof: 🎥 video: Contributor real behavior proof includes video or recording evidence. Mantis Telegram Desktop proof shows the baseline Media failed result and the candidate delivering an .html attachment; local test output is supplemental.
mantis: telegram-visible-proof: Mantis should capture Telegram visible proof. The PR changes visible Telegram generated-report attachment delivery, which is exactly the kind of behavior a short Telegram Desktop proof can demonstrate.

Evidence reviewed

PR surface:

Source +79, Tests +103. Total +182 across 2 files.

View PR surface stats

Area	Files	Added	Removed	Net
Source	1	87	8	+79
Tests	1	104	1	+103
Docs	0	0	0	0
Config	0	0	0	0
Generated	0	0	0	0
Other	0	0	0	0
Total	2	191	9	+182

Security concerns:

[medium] Pre-read path trust can go stale — src/media/web-media.ts:990
The PR approves the pathname before reading and later allows HTML based on that stale flag, so a swap between approval and read can bypass the outside-root/symlink/hardlink protections.
Confidence: 0.86

What I checked:

Current main fails closed for host-read HTML: Current main treats extension-declared .html as a declared text MIME but throws path-not-allowed unless it is CSV/Markdown, which matches the reported regression surface. (src/media/web-media.ts:278, ba2620a9afa9)
Current main test documents the blocked behavior: The existing test asserts host-read HTML is rejected without a separate security-boundary approval. (src/media/web-media.test.ts:653, ba2620a9afa9)
PR head checks path trust before reading: The PR computes trustedGeneratedHtmlPath from isTrustedGeneratedHostReadHtmlPath(mediaUrl) before calling the configured read function on the same pathname. (src/media/web-media.ts:988, aef79032c04a)
PR head reuses the pre-read trust flag: After the bytes are read and sniffed, the allow decision receives the earlier trustedGeneratedHtmlPath flag instead of proving that the opened file is still the approved file. (src/media/web-media.ts:1030, aef79032c04a)
PR adds focused regression tests: The PR adds tests for allowed generated HTML, outside-root HTML, symlink, hardlink, and non-HTML .html cases, but those tests do not exercise a path swap between approval and read. (src/media/web-media.test.ts:654, aef79032c04a)
Mantis proof covers the visible Telegram behavior: The prepared ffprobe/contact-sheet artifacts and Mantis comment show baseline Telegram delivery ending in Media failed and the candidate delivering an .html attachment.

Likely related people:

steipete: Recent current-main history and blame show substantial work on src/media/web-media.ts, including the current line set and earlier host media read capability plumbing. (role: recent area contributor; confidence: high; commits: c3e02d9fd454, 3bb02d333865; files: src/media/web-media.ts, src/media/web-media.test.ts, src/media/load-options.ts)
mjamiv: The related merged media-staging PR intentionally kept host-read HTML denied pending this separate security-boundary review. (role: related regression PR author; confidence: medium; commits: f22c3a518e4d; files: src/media/web-media.ts, src/media/web-media.test.ts, src/auto-reply/reply/reply-media-paths.test.ts)
Chen Chia Yang: Introduced the host-local CSV/Markdown upload handling and focused tests that this PR extends for trusted generated HTML. (role: host-read media feature contributor; confidence: medium; commits: d2a219ea44d9; files: src/media/web-media.ts, src/media/web-media.test.ts)

What the crustacean ranks mean

🦀 challenger crab: rare, exceptional readiness with strong proof, clean implementation, and convincing validation.
🦞 diamond lobster: very strong readiness with only minor maintainer review expected.
🐚 platinum hermit: good normal PR, likely mergeable with ordinary maintainer review.
🦐 gold shrimp: useful signal, but proof or patch confidence is still limited.
🦪 silver shellfish: thin signal; proof, validation, or implementation needs work.
🧂 unranked krab: not merge-ready because proof is missing/unusable or there are serious correctness or safety concerns.
🌊 off-meta tidepool: rating does not apply to this item.

Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics.

How this review workflow works

ClawSweeper keeps one durable marker-backed review comment per issue or PR.
Re-runs edit this comment so the latest verdict, findings, and automation markers stay together instead of adding duplicate bot comments.
A fresh review can be triggered by eligible @clawsweeper re-review comments, exact-item GitHub events, scheduled/background review runs, or manual workflow dispatch.
PR/issue authors and users with repository write access can comment @clawsweeper re-review or @clawsweeper re-run on an open PR or issue to request a fresh review only.
Maintainers can also comment @clawsweeper review to request a fresh review only.
Fresh-review commands do not start repair, autofix, rebase, CI repair, or automerge.
Maintainer-only repair and merge flows require explicit commands such as @clawsweeper autofix, @clawsweeper automerge, @clawsweeper fix ci, or @clawsweeper address review.
Maintainers can comment @clawsweeper explain to ask for more context, or @clawsweeper stop to stop active automation.

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9d8d0a61f3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

chatgpt-codex-connector · 2026-05-29T10:14:45Z

+  const trustedGeneratedHtmlPath =
+    hostReadDeclaredMime === "text/html"
+      ? await isTrustedGeneratedHostReadHtmlPath(mediaUrl)


Bind trusted HTML checks to the bytes read

When a .html host-read path is approved here, the code later calls the supplied readFile using the same pathname without revalidating that the opened file is still the single-link regular file under the OpenClaw temp root. In the scenario this change is trying to protect (symlink/hardlink/outside-root HTML), anything that can modify the temp path between this check and the read can swap the approved file for a symlink or hardlink to an outside HTML file and still pass isTrustedGeneratedHostReadHtml because the stale trustedGeneratedHtmlPath flag is reused. The trust decision should be tied to the opened file/realpath after read, or the read should be performed through a safe open that rejects swaps.

Useful? React with 👍 / 👎.

openclaw-mantis · 2026-05-29T10:23:02Z

Mantis Telegram Desktop Proof

Summary: Mantis captured native Telegram Desktop before/after GIFs for generated HTML report delivery.

Scenario: telegram-desktop-proof
Trigger: pull_request_target
Run: https://github.com/openclaw/openclaw/actions/runs/26630292110
Artifact: https://github.com/openclaw/openclaw/actions/runs/26630292110/artifacts/7288478151
Baseline: pass at 7d76e54f2bbda91d848d92dc72324dd9f6104f04, expected baseline visual proof captured
Candidate: pass at 0f2e690fd6c8e0d637787c86cee61282b32e64ef, expected candidate visual proof captured
Overall: true

Main screenshot	This PR screenshot

Main	This PR

Motion-trimmed clips:

Raw QA files: https://artifacts.openclaw.ai/mantis/telegram-desktop/pr-87982/run-26630292110-1/index.json

…026.5.28) (#759) This PR contains the following updates: | Package | Update | Change | |---|---|---| | [ghcr.io/openclaw/openclaw](https://openclaw.ai) ([source](https://github.com/openclaw/openclaw)) | patch | `2026.5.27` → `2026.5.28` | --- ### Release Notes <details> <summary>openclaw/openclaw (ghcr.io/openclaw/openclaw)</summary> ### [`v2026.5.28`](https://github.com/openclaw/openclaw/blob/HEAD/CHANGELOG.md#2026528) [Compare Source](openclaw/openclaw@v2026.5.27...v2026.5.28) ##### Highlights - Agent and Codex runtime recovery is steadier: subagents keep cwd/workspace separation, hook context stays prompt-local, session locks release on timeout abort while live OpenClaw locks survive cleanup, stale restart continuations are avoided, and Codex app-server/helper failures no longer tear down shared runtime state. ([#87218](openclaw/openclaw#87218), [#86875](openclaw/openclaw#86875), [#87409](openclaw/openclaw#87409), [#87399](openclaw/openclaw#87399), [#87375](openclaw/openclaw#87375), [#88129](openclaw/openclaw#88129)) - Channel delivery and session identity got safer across outbound plugin hooks, Matrix room ids, iMessage reactions/approvals, Slack final replies, Discord recovered tool warnings, runtime-config message actions, WhatsApp profile auth roots, Telegram polling, and Microsoft Teams service URL trust checks. ([#73706](openclaw/openclaw#73706), [#75670](openclaw/openclaw#75670), [#87366](openclaw/openclaw#87366), [#87451](openclaw/openclaw#87451), [#87334](openclaw/openclaw#87334), [#84535](openclaw/openclaw#84535), [#82492](openclaw/openclaw#82492), [#83304](openclaw/openclaw#83304), [#87160](openclaw/openclaw#87160)) - Mobile and chat surfaces got a broader refresh: the iOS Pro UI, hosted push relay default, realtime Talk tab playback, Gateway chat transport, onboarding, Talk permissions, WebChat reconnect delivery, and session picker behavior now preserve more state across reconnects and empty searches. ([#87367](openclaw/openclaw#87367), [#87531](openclaw/openclaw#87531), [#87682](openclaw/openclaw#87682), [#88096](openclaw/openclaw#88096), [#88105](openclaw/openclaw#88105)) Thanks [@ngutman](https://github.com/ngutman) and [@BunsDev](https://github.com/BunsDev). - Browser, channel, and automation inputs are stricter: Browser tool timeouts, viewport/tab indices, Gateway ports, cron retry handling, Discord component ids, schema array refs, Telegram callback pages, and channel progress callbacks now reject malformed values earlier and preserve the intended delivery context. ([#82887](openclaw/openclaw#82887)) - Provider, media, and document coverage expands with Claude Opus 4.8, Fal Krea image schemas, NVIDIA featured models, MiniMax streaming music responses, encrypted PDF extraction, voice model catalogs, GitHub Copilot agent runtime support, and a Codex Supervisor plugin path for delegated Codex workflows. ([#87845](openclaw/openclaw#87845), [#87890](openclaw/openclaw#87890), [#80775](openclaw/openclaw#80775), [#84764](openclaw/openclaw#84764), [#87751](openclaw/openclaw#87751), [#87794](openclaw/openclaw#87794)) - CLI, auth, doctor, and provider paths fail faster and recover more clearly: malformed numeric/version options are rejected, workspace dotenv provider credentials are ignored, heartbeat defaults, OAuth/token lifetimes, and local service startup requests are bounded, agent auth health labels are clearer, legacy `api_key` auth profiles migrate to canonical form, and restart guidance is actionable. ([#87398](openclaw/openclaw#87398), [#86281](openclaw/openclaw#86281), [#87361](openclaw/openclaw#87361), [#88133](openclaw/openclaw#88133), [#83655](openclaw/openclaw#83655), [#87559](openclaw/openclaw#87559), [#88088](openclaw/openclaw#88088), [#85924](openclaw/openclaw#85924)) Thanks [@vincentkoc](https://github.com/vincentkoc) and [@giodl73-repo](https://github.com/giodl73-repo). - Plugin and Gateway hot paths do less repeated work while preserving cache correctness for install records, config JSON parsing, tool search catalogs, session stores, manifest model rows, auto-enabled plugin config, browser tokens, viewer assets, and release-split external plugin packages. ([#86699](openclaw/openclaw#86699)) - Release, QA, and E2E validation now bound more log, artifact, harness, and cross-OS waits so failing lanes produce proof instead of hanging or false-greening. ##### Changes - Status: show active subagent details in status output. - Diffs: split the default language pack and expand default Diffs language coverage while keeping the host floor aligned. ([#87370](openclaw/openclaw#87370), [#87372](openclaw/openclaw#87372)) Thanks [@RomneyDa](https://github.com/RomneyDa). - ClawHub: add plugin display names plus skill verification and trust surfaces. ([#87354](openclaw/openclaw#87354), [#86699](openclaw/openclaw#86699)) Thanks [@thewilloftheshadow](https://github.com/thewilloftheshadow) and [@Patrick-Erichsen](https://github.com/Patrick-Erichsen). - iOS: refresh the dev app with Pro Command, Chat, Agents, Settings, hosted push relay defaults, and realtime Talk playback wired to gateway sessions, diagnostics, chat, and realtime Talk. ([#87367](openclaw/openclaw#87367), [#88096](openclaw/openclaw#88096), [#88105](openclaw/openclaw#88105)) Thanks [@Solvely-Colin](https://github.com/Solvely-Colin) and [@ngutman](https://github.com/ngutman). - Docs: clarify Codex computer-use setup, paste-token stdin auth setup, macOS gateway sleep troubleshooting, native Codex hook relay recovery, container model auth, install deployment cards, device-token admin gating, CLI setup flow compatibility, Notte cloud browser CDP setup, and backport targets. ([#87313](openclaw/openclaw#87313), [#63050](openclaw/openclaw#63050), [#87685](openclaw/openclaw#87685)) Thanks [@bdjben](https://github.com/bdjben), [@liaoandi](https://github.com/liaoandi), and [@thewilloftheshadow](https://github.com/thewilloftheshadow). - PDF/tools: use ClawPDF for PDF extraction, support encrypted PDF extraction, and surface MCP structured content in agent tool results. ([#87670](openclaw/openclaw#87670), [#87751](openclaw/openclaw#87751)) - Providers: add Claude Opus 4.8 support, Fal Krea image model schemas, NVIDIA featured model catalogs, MiniMax streaming music responses, and provider-backed voice model catalogs. ([#87845](openclaw/openclaw#87845), [#87890](openclaw/openclaw#87890), [#80775](openclaw/openclaw#80775), [#84764](openclaw/openclaw#84764), [#87794](openclaw/openclaw#87794)) Thanks [@eleqtrizit](https://github.com/eleqtrizit) and [@vincentkoc](https://github.com/vincentkoc). - Codex/GitHub: add the GitHub Copilot agent runtime and the Codex Supervisor plugin package. - Plugins: externalize GitHub Copilot and Tokenjuice as official install-on-demand plugins with npm and ClawHub publish metadata. - Workboard: add agent coordination tools for tracking and handing off active agent work. - Discord: show commentary in progress drafts so live Discord runs expose useful in-progress context. ([#85200](openclaw/openclaw#85200)) - Plugin SDK: add a reply payload sending hook for plugins that need to deliver channel-owned replies and flatten package types for SDK declarations. ([#82823](openclaw/openclaw#82823), [#87165](openclaw/openclaw#87165)) Thanks [@piersonr](https://github.com/piersonr) and [@RomneyDa](https://github.com/RomneyDa). - Policy: add policy comparison, ingress-channel conformance, and sandbox-posture conformance checks. ([#85572](openclaw/openclaw#85572), [#85744](openclaw/openclaw#85744), [#86768](openclaw/openclaw#86768)) ##### Fixes - Agents: fall back to local config pruning when the optional `agents delete` Gateway probe cannot authenticate, so offline installs can still delete agents without removing shared workspaces. - Tighten phone-control mutation authorization \[AI]. ([#87150](openclaw/openclaw#87150)) Thanks [@pgondhi987](https://github.com/pgondhi987). - Clarify directive persistence authorization policy \[AI]. ([#86369](openclaw/openclaw#86369)) Thanks [@pgondhi987](https://github.com/pgondhi987). - Agents/Codex: keep spawned agent cwd/workspace state separated, forward ACP spawn attachments, keep hook context prompt-local, release session locks on timeout abort and runtime teardown without deleting live OpenClaw-owned locks during cleanup, avoid session event queue self-wait, clean up exec abort listeners, stream assistant deltas incrementally, recover raw missing-thread compaction failures, preserve rotated compaction session identity, keep compaction-timeout snapshots continuable, preserve shared app-server state across startup or helper failures, keep native hook relay alive across restarts and prune stale bridge files, close native hook relay replacement races, keep Claude live tool progress visible for watchdog recovery, suppress abandoned requester completion handoff, route workspace memory through tools, resolve Codex runtime models first, report quarantined dynamic tools, format `skills` command output, bind node auto-review to prepared plans, retry Claude CLI transcript probes, and bound compaction/steering retries. ([#87218](openclaw/openclaw#87218), [#86875](openclaw/openclaw#86875), [#86123](openclaw/openclaw#86123), [#88129](openclaw/openclaw#88129), [#87399](openclaw/openclaw#87399), [#87375](openclaw/openclaw#87375), [#72574](openclaw/openclaw#72574), [#87383](openclaw/openclaw#87383), [#87400](openclaw/openclaw#87400), [#83022](openclaw/openclaw#83022), [#87671](openclaw/openclaw#87671), [#87738](openclaw/openclaw#87738), [#87747](openclaw/openclaw#87747), [#87706](openclaw/openclaw#87706), [#87546](openclaw/openclaw#87546), [#87541](openclaw/openclaw#87541), [#81048](openclaw/openclaw#81048)) Thanks [@mbelinky](https://github.com/mbelinky), [@Alix-007](https://github.com/Alix-007), [@luoyanglang](https://github.com/luoyanglang), [@yetval](https://github.com/yetval), [@sjf](https://github.com/sjf), [@joshavant](https://github.com/joshavant), [@benjamin1492](https://github.com/benjamin1492), [@c19354837](https://github.com/c19354837), [@fuller-stack-dev](https://github.com/fuller-stack-dev), [@pfrederiksen](https://github.com/pfrederiksen), and [@dodge1218](https://github.com/dodge1218). - Codex Supervisor: keep real-home app-server MCP session listing on the loaded state path, bound stored history scans, and close WebSocket probes cleanly. - Channels: thread canonical session keys into outbound hooks, preserve Matrix room-id case, keep fallback tool warnings mention-inert, retain delivered Slack final replies during late cleanup, continue iMessage polling after denied reactions, suppress duplicate native exec approvals, resolve Gateway message actions against the active runtime config, preserve Telegram SecretRef prompt config and polling keepalives, preserve WhatsApp profile auth roots, QR display, document filenames, and plugin hook config, suppress Discord recovered tool warnings, preserve the Discord voice outbound helper, cap Discord/Signal/Zalo channel request and container timeouts, and block untrusted Teams service URLs while keeping TeamsSDK patterns aligned. ([#73706](openclaw/openclaw#73706), [#75670](openclaw/openclaw#75670), [#87366](openclaw/openclaw#87366), [#87451](openclaw/openclaw#87451), [#87465](openclaw/openclaw#87465), [#87334](openclaw/openclaw#87334), [#84535](openclaw/openclaw#84535), [#76262](openclaw/openclaw#76262), [#83304](openclaw/openclaw#83304), [#82492](openclaw/openclaw#82492), [#87581](openclaw/openclaw#87581), [#77114](openclaw/openclaw#77114), [#86426](openclaw/openclaw#86426), [#85529](openclaw/openclaw#85529), [#87160](openclaw/openclaw#87160)) Thanks [@zeroaltitude](https://github.com/zeroaltitude), [@lukeboyett](https://github.com/lukeboyett), [@jarvis-mns1](https://github.com/jarvis-mns1), [@xiaotian](https://github.com/xiaotian), [@funmerlin](https://github.com/funmerlin), [@joshavant](https://github.com/joshavant), [@eleqtrizit](https://github.com/eleqtrizit), [@heyitsaamir](https://github.com/heyitsaamir), [@amittell](https://github.com/amittell), [@lidge-jun](https://github.com/lidge-jun), [@liorb-mountapps](https://github.com/liorb-mountapps), [@masatohoshino](https://github.com/masatohoshino), [@bladin](https://github.com/bladin), and [@giodl73-repo](https://github.com/giodl73-repo). - CLI/auth/doctor/providers: reject malformed numeric/timeout/subcommand-version inputs, ignore workspace dotenv provider credentials, wait for respawn child shutdown, bound heartbeat defaults plus Codex, GitHub Copilot, OpenAI, Anthropic, Google, Feishu, LM Studio, MiniMax, Xiaomi TTS, and local-provider OAuth/token/model requests, harden Codex auth probes, label auth health by agent, preserve explicit agentRuntime pins during Codex model migration, warm provider auth off the main thread, honor Codex response timeouts, stop migrating current Claude Haiku 4.5 profiles to Sonnet, bound local service startup, resolve GPT-5.5 without cached catalog, migrate legacy memory auto-provider config, rewrite non-canonical `api_key` auth profiles, and make doctor restart follow-ups actionable. ([#87398](openclaw/openclaw#87398), [#86281](openclaw/openclaw#86281), [#87361](openclaw/openclaw#87361), [#88133](openclaw/openclaw#88133), [#83655](openclaw/openclaw#83655), [#87559](openclaw/openclaw#87559), [#87719](openclaw/openclaw#87719), [#88088](openclaw/openclaw#88088), [#85924](openclaw/openclaw#85924), [#84362](openclaw/openclaw#84362)) Thanks [@Patrick-Erichsen](https://github.com/Patrick-Erichsen), [@samzong](https://github.com/samzong), [@giodl73-repo](https://github.com/giodl73-repo), [@alkor2000](https://github.com/alkor2000), [@mmaps](https://github.com/mmaps), [@nxmxbbd](https://github.com/nxmxbbd), and [@vincentkoc](https://github.com/vincentkoc). - Gateway/security/session state: expire browser tokens after auth rotation, scope assistant idempotency dedupe, drain probe client closes, avoid stale restart continuation reuse, preserve retry-after fallbacks and stale rate-limit cooldown probes, bound webchat image and artifact transcript scans, include seconds in inbound metadata timestamps, clear completed session active runs, clear stale chat stream buffers, and evict current plugin-state namespaces at row caps. ([#87810](openclaw/openclaw#87810), [#87833](openclaw/openclaw#87833), [#75089](openclaw/openclaw#75089)) Thanks [@joshavant](https://github.com/joshavant) and [@litang9](https://github.com/litang9). - Config/parsing/network: reject partial numeric parsing, parse provider/Discord retry headers and dates strictly, honor IPv6 and bare IPv6 `no_proxy` entries, preserve empty plugin allowlists, canonicalize secret target array indexes, and reject malformed media content lengths, inspected TCP ports, marketplace content lengths, cron epochs, sandbox stat fields, unsafe duration values, empty config path segments, noncanonical schema array refs, unsafe Telegram callback pages, and invalid Teams attachment-fetch DNS targets. ([#87883](openclaw/openclaw#87883)) Thanks [@zhangguiping-xydt](https://github.com/zhangguiping-xydt). - Browser/input hardening: reject invalid tab indexes, excessive viewport resizes, explicit zero CDP ports, malformed geolocation options, unsafe screenshot or permission-grant timeouts, loose response-body limits, invalid cookie expiries, and non-finite Browser tool delays/timeouts. - Cron/automation: retry recurring jobs after transient model rate limits before waiting for the next scheduled slot, and preflight model fallbacks before skipping scheduled work. ([#82887](openclaw/openclaw#82887)) Thanks [@chen-zhang-cs-code](https://github.com/chen-zhang-cs-code). - Auto-reply/directives: respect provider and relayed channel metadata during directive persistence so channel-originated decisions keep their intended context. ([#87683](openclaw/openclaw#87683)) - WhatsApp: resolve the auth directory from the active profile so profile-scoped WhatsApp installs do not drift to the wrong credential root. ([#82492](openclaw/openclaw#82492)) Thanks [@lidge-jun](https://github.com/lidge-jun). - Gateway/session state: clear completed session active runs, avoid cold-loading providers for MCP inventory, cache single-session child indexes, cap handshake timers, and bound preauth, auth-guard, media, transcript, readiness, and port options. - Channels/replies: preserve channel-owned progress callbacks when verbose output is off, keep group-room progress suppression intact, prefer external session delivery context, escape Discord component id delimiters, force final TUI chat repaints, show Slack reasoning previews, and normalize Discord/Matrix/Mattermost channel numeric options. ([#87476](openclaw/openclaw#87476), [#87423](openclaw/openclaw#87423)) - Agents/tool args: harden smart-quoted argument repair for edit arrays and exact escaped arguments so model-produced tool calls recover without corrupting valid input. ([#86611](openclaw/openclaw#86611)) Thanks [@ferminquant](https://github.com/ferminquant). - Providers/agents: preserve seeded Anthropic signatures, preserve signed thinking payloads, concatenate signature-delta chunks, preserve DeepSeek `reasoning_content` replay across tier suffixes, apply OpenRouter strict9 ids to Mistral routes, promote Ollama plain-text tool calls, load NVIDIA featured model catalogs, stream MiniMax music generation responses, and recover empty preflight compaction. ([#87593](openclaw/openclaw#87593), [#87493](openclaw/openclaw#87493), [#80775](openclaw/openclaw#80775), [#84764](openclaw/openclaw#84764)) Thanks [@Pluviobyte](https://github.com/Pluviobyte) and [@eleqtrizit](https://github.com/eleqtrizit). - Media/images: skip CLI image cache refs when resolving generated images, allow trusted generated HTML attachments, and bound generated video downloads so stale refs and slow providers fail cleanly. ([#87523](openclaw/openclaw#87523), [#87982](openclaw/openclaw#87982)) - File transfer: handle late tar stdin pipe errors after archive validation or unpacking has already settled. - Performance: trust install-record caches between reloads, prefer native JSON parsing, reuse unchanged tool-search catalogs, reuse gateway session and plugin metadata paths, skip unchanged store serialization, patch single-entry session writes, add precomputed session patch writers, reduce store clone allocations, cache manifest model catalog rows and auto-enabled plugin config, avoid full session snapshots for entry reads, defer configured Slack full startup, prefer bundled plugin dist entries, and slim current metadata identity caches. ([#87760](openclaw/openclaw#87760)) - Docker/release/QA: package runtime workspace templates, stream cross-OS served artifacts, preserve sparse Crabbox run artifacts, isolate npm plugin installs per package, reject incompatible package plugin API installs, drop the leftover root Sharp dependency from package manifests after the Rastermill migration, bound OpenClaw instance logs, plugin gauntlet relay logs, MCP channel buffers, kitchen-sink scans, agent-turn assertions, QA-Lab credential broker calls, QA Matrix substrate requests, and release scenario logs, and keep release/google live guards current. ([#87647](openclaw/openclaw#87647), [#87477](openclaw/openclaw#87477)) Thanks [@rohitjavvadi](https://github.com/rohitjavvadi) and [@vincentkoc](https://github.com/vincentkoc). - Release/CI: bound manual git fetches, ClawHub verifier responses, ClawHub owner metadata, dependency-guard error bodies, Parallels limits, startup/test/memory budget parsing, and diffs viewer build warnings so release lanes fail with useful proof instead of hanging. ([#87839](openclaw/openclaw#87839)) </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about these updates again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).  Reviewed-on: https://git.erwanleboucher.dev/eleboucher/homelab/pulls/759

openclaw-barnacle Bot added size: M maintainer Maintainer-authored PR labels May 29, 2026

clawsweeper Bot temporarily deployed to qa-live-shared May 29, 2026 09:47 Inactive

clawsweeper Bot added rating: 🧂 unranked krab Not merge-ready due to missing proof or serious correctness/safety concerns. and removed rating: 🦪 silver shellfish Thin PR readiness signal; proof, validation, or implementation needs work. labels May 29, 2026

steipete force-pushed the fix/telegram-generated-html-attachments branch from 6c5e368 to 9d8d0a6 Compare May 29, 2026 10:12

chatgpt-codex-connector Bot reviewed May 29, 2026

View reviewed changes

fix(media): allow trusted generated html attachments

aef7903

steipete force-pushed the fix/telegram-generated-html-attachments branch from 9d8d0a6 to aef7903 Compare May 29, 2026 10:23

steipete merged commit 68d0c0f into main May 29, 2026
185 of 188 checks passed

steipete deleted the fix/telegram-generated-html-attachments branch May 29, 2026 10:33

github-actions Bot mentioned this pull request May 29, 2026

📡 Upstream Digest — 2026-05-29 11:21 UTC curtismercier/openclaw-mods#970

Open

github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request May 30, 2026

fix(media): allow trusted generated html attachments (openclaw#87982)

56a5ba4

SYU8384 pushed a commit to SYU8384/openclaw that referenced this pull request Jun 3, 2026

fix(media): allow trusted generated html attachments (openclaw#87982)

cb7c74d

This was referenced Jun 5, 2026

[Bug]: Telegram outbound rejects trusted generated HTML after staging strips trusted-root marker (PR #87982 missed e2e case) #90557

Open

fix: allow trusted host-read html after outbound staging #90579

Open

sablehead pushed a commit to sablehead/openclaw that referenced this pull request Jun 10, 2026

fix(media): allow trusted generated html attachments (openclaw#87982)

bd7dcbe

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix(media): allow trusted generated html attachments#87982

fix(media): allow trusted generated html attachments#87982
steipete merged 1 commit into
mainfrom
fix/telegram-generated-html-attachments

steipete commented May 29, 2026 •

edited

Loading

Uh oh!

clawsweeper Bot commented May 29, 2026 •

edited

Loading

Uh oh!

chatgpt-codex-connector Bot left a comment

Uh oh!

chatgpt-codex-connector Bot May 29, 2026

Uh oh!

openclaw-mantis Bot commented May 29, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

steipete commented May 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Verification

Real behavior proof

Uh oh!

clawsweeper Bot commented May 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

chatgpt-codex-connector Bot May 29, 2026

Choose a reason for hiding this comment

Uh oh!

openclaw-mantis Bot commented May 29, 2026

Mantis Telegram Desktop Proof

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

steipete commented May 29, 2026 •

edited

Loading

clawsweeper Bot commented May 29, 2026 •

edited

Loading