fix(ocsf): attribute MXC proxy events to sandboxes - #3434
prekshivyas wants to merge 1 commit into
Conversation
NVBug 6783086 Signed-off-by: Prekshi Vyas <prekshiv@nvidia.com>
8169e22 to
ed94456
Compare
|
Live Windows ARM64 MXC qualification completed against the current
This live run confirms both the original failure and the fix. The unchanged actor PID 0 behavior remains outside this PR's scope, as noted in the description. |
Qualification before/after summaryNVBug: 6783086 — MXC proxy OCSF events can be attributed to the wrong sandbox Before: Host-proxy audit decisions depended on process-global fallback context. With multiple sandboxes under one gateway, denied CONNECT/forward-proxy events could lose or misattribute sandbox ID/name. After: Each Windows MXC host proxy carries an immutable per-sandbox Qualification status: Focused scenario PASS; full current-tip qualification pending. Rebase onto the current |
Summary
Give every Windows MXC host proxy its own immutable OCSF event context so denied HTTP/network events retain the sandbox ID and name when one gateway hosts multiple sandboxes. This removes the incorrect dependency on the process-global fallback context for top-level CONNECT and forward-proxy decisions.
Related Issue
NVBug 6783086 (internal)
Changes
EventContextfromHostProxyConfigwindowsbranchTesting
mise run pre-commitpasses — blocked on Windows ARM64 becausegrpcio-tools 1.78.0invokes MSVC with incompatible/std:c++17and/std:c11flags; attempted with the repository Python 3.14 and an isolated Python 3.13 environmentcargo check -p openshell-supervisor-network --libcargo clippy -p openshell-supervisor-network --all-targets --no-deps --target aarch64-pc-windows-msvccargo test -p openshell-supervisor-network— 1,230 passed, 2 ignored, and 3 pre-existing Windows failures caused by an unescaped executable path in an MCP YAML fixtureChecklist
Scope note
This PR fixes
container.uidandcontainer.nameattribution. The static Windows proxy identity still has no trustworthy per-process PID, so the existing actor PID behavior is unchanged.