Repository navigation
Draft implementation: managed outbound HTTPS for dedicated Gateways - #2518
miguelbranco80 wants to merge 39 commits into
Conversation
|
🦞👀 Pull request received. I will update this pull request when review starts. ClawSweeper review blockedAutomated review did not run, so no review verdict was produced. Reason: The input-safety check rejected material in this revision. No detected value, path, or scanner output is reproduced here. Automatic review is on hold, including after source changes. Request a fresh re-review after resolving the failure; maintainers can also release the hold through an intentional scanner-policy update. Next step: If this is a genuine credential, remove and rotate it. If it is an intentional test fixture, a maintainer must review and qualify it. |
|
Free's clanker here (an AI agent working on Free's behalf). I review fork PR runs for CI-safety before approving them. I have not approved the runs for e2dc302 on this PR, because the PR changes CI behaviour, and a maintainer has to approve that kind of change by hand:
I found nothing that reads tokens or secrets or calls outside hosts, and |
|
Free's clanker here (an AI agent working on Free's behalf). Heads-up about a conflict coming from main: #3111 moved the body of This PR edits the old body of |
The RFC has been extracted to #3520 for easier architectural review and discussion. This PR remains a draft implementation for reference and experimentation.
The RFC file is identical to the focused proposal. The implementation plan retains delivery details. This synchronization changes documentation only; it adds no runtime behavior or qualification claim. Documentation length, links, specification metadata, formatting, workspace, and diff checks passed.
Verification
17340527e83e97f5bfa77220cb7f04ee6124bb65passed local typecheck, lint, formatting, workspace, OpenAPI, documentation and focused API/proxy/browser checks. CI Required and each selected required lane passed, including PostgreSQL and Kubernetes fixture workflows. First Agent Smoke passed. CI tested synthetic merge58d5bec339d47b5bb036eb1adf61bf8168cf75c3, not the locally reviewed pinned-base tree; native container verification was skipped. These results do not qualify a later head.531fa5ed4, OpenClaw90d30a1178a79dddd92e6190b66b95d89dfb3ca8, macOS arm64, passed restricted/public plugin fetch, real model turns, network denials, cutover, held-tunnel withdrawal, worker restart and policy tightening in enforcing k3d with PostgreSQL. The public-reader response was an HTTP fixture. This is historical, not current-runtime or live-provider qualification.58822505a, Storybook assetefa81d40bf9fa7d1, Chromium, desktop 1440px/mobile 390px; ninePages/Agent detail/GatewayNetwork*stories plus Access. Saved state, dirty/deploy-disabled state, mobile, 18-second walkthrough. These fixtures do not prove persistence, deployment, credential propagation or native connectivity; final-source visual proof remains unqualified.Review and qualification holds
msteamsConfiguration, but bundled automatic requirements cover Slack;channelsOnlycan admit no route. Assess compatibility with existing upstream behavior, not a new Teams delivery requirement.sessions.createbuild timeout; fresh selected model/proxy proof; full Helm-installed production qualification; isolated live-provider and deterministic mixed-answer/rebinding proof; Protected Harness/Credential Proxy composition; inherited local OpenShell IPv6 TLS failure. Historical evidence and later reviews with no new findings do not release these holds.