feat(cli): openlock logs surfaces L7 egress headers via --debug-egress (openlock-dlv Tier 2)#69
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Completes openlock-dlv Tier 2: an opt-in
--debug-egressflag that surfaces the in-container proxy's L7 request/response headers throughopenlock logs— for billing-bucket / egress verification. Off by default; no header logging in normal runs.Pairs with fork v0.6.6 (released): the supervisor's debug-gated header logging +
--log-levelonopenshell sandbox create+ file-layer un-pin. This PR bumpsOPENSHELL_FORK_TAGto v0.6.6.Changes
--debug-egressonopenlock sandbox→ threadsdebugEgressthroughSandboxOpts→createSession→buildOpenshellCreateArgv, which appends--log-level debugtoopenshell sandbox create. The fork then setsSandboxSpec.log_level→OPENSHELL_LOG_LEVEL→ supervisor at DEBUG → file layer carries the header lines thatopenlock logstails.OPENSHELL_FORK_TAGv0.6.5→v0.6.6(the release carrying the fork-side logging +--log-level).Wire
openlock sandbox --debug-egress→openshell sandbox create --log-level debug→SandboxSpec.log_level→OPENSHELL_LOG_LEVEL→ supervisor DEBUG → un-pinned file layer →openlock logs <name>.Testing
bun run lint && typecheck && test (839) && knip— all green.buildOpenshellCreateArgvappends--log-level debugiffdebugEgress, omits it otherwise.anthropic-beta: …,oauth-2025-04-20,…andanthropic-ratelimit-unified-overage-status: rejectedviaopenlock logs— reproducing the dkf/mitmproxy base-bucket finding in-sandbox.Notes