Skip to content

Let microsandbox domain allows pass HTTPS - #894

Merged
SaladDay merged 1 commit into
mainfrom
codex/msb-domain-https
Oct 27, 2026
Merged

SaladDay merged 1 commit into
mainfrom
codex/msb-domain-https

Conversation

@SaladDay

@SaladDay SaladDay commented Oct 27, 2026 •

Copy link
Copy Markdown
Collaborator

Problem

On microsandbox, every sandbox whose egress is not fully public could not reach its model provider: the Turn failed with connection_failed. Restricted domains failed the same way over HTTPS.

Reproduced with the adapter's rule order and the v0.7.8 runtime: the SYN to the allowed domain's address passes, then the connection is reset after the ClientHello. The runtime logs TCP egress denied by strict hostname policy sni="<allowed domain>". microsandbox's strict mode, on by default, accepts a domain allow over TLS only when TLS interception can inspect the authority.

Change

The adapter sets DisableStrict. Domain allows then match by the guest's DNS answer and the TLS server name. Group, address and default-deny rules are unchanged.

Verification

Same rules, same runtime, before and after:

Probe strict (before) strict off (after)
HTTPS to the allowed domain, port-limited rule reset at TLS HTTP 404 from the server
HTTPS to the allowed domain, port-less rule reset at TLS HTTP 404 from the server
HTTPS to a domain not allowed refused refused

go vet and the adapter's network tests pass.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

@SaladDay
SaladDay merged commit 297c889 into main Oct 27, 2026
18 checks passed
@SaladDay
SaladDay deleted the codex/msb-domain-https branch October 27, 2026 19:40
microsandbox's strict hostname mode, on by default, accepts a domain allow
over TLS only with TLS interception. Every domain rule of a not-public
egress (the model endpoint, restricted domains, public domains of the cap)
therefore reset at the TLS handshake. Turn it off so these rules match by
the guest's DNS answer and the TLS server name.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant