openclaw - 💡(How to fix) Fix test(plugin-sdk): track origin-trust SSRF edge coverage and docs [1 pull requests]

Official PRs (…)
ON THIS PAGE

Recommended Tools

×6

Utilities matched from this issue’s tags and category — try them while you read without losing context.

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

Helpful · Quick feedback

Loading…

Fix Action

Fixed

RAW_BUFFERClick to expand / collapse

Follow-up from #80751 multi-lane review. This is not a blocker for #80751.

#80751 now pins the SDK boundary smoke case: plugin-supplied allowedOrigins entries for metadata/link-local targets, and DNS rebinding from a trusted private origin to metadata IPs, still get rejected by the resolver.

Remaining non-blocking hardening items:

  • Add origin-normalization assertions for custom provider baseUrl: trailing slash equivalence, default-port elision, and userinfo stripping via URL.origin.
  • Add a concurrent multi-agent origin-trust isolation test for overlapping but distinct private baseUrls.
  • Document plugin SDK origin-scoped trust semantics in docs/plugins/sdk-subpaths.md: request origin must match exactly, hostname promotion applies only to the current request, and metadata/link-local blocking remains enforced by the resolver.
  • Consider adding an invariant comment for IPv6 zone-id literals being fail-closed in the loose IP parser and DNS-answer re-check.

The BLOCKED_HOSTNAMES symmetry and fakeIpPolicy endpoint-class gating notes from the review are intentionally left as maintainer-direction questions on #80751 rather than bundled into this follow-up.

Vote matrix · Quick signals

Works
Did the solution work? Tap to confirm.
Easy Fix
Was it a quick fix?
Time Saver
Did it save you time?
Blocking
Was it severely blocking?
Common Issue
Are others likely hitting this too?
Flaky / Intermittent
Is it intermittent?
Verified / Reproducible
Can you reproduce it reliably?
Loading…

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Back to top recommendations

TRENDING

openclaw - 💡(How to fix) Fix test(plugin-sdk): track origin-trust SSRF edge coverage and docs [1 pull requests]