Skip to content

Conversation

@mishushakov
Copy link
Member

@mishushakov mishushakov commented Nov 26, 2025

Note

Add E2B-Traffic-Access-Token header support to JS/Python clients and introduce secure-traffic test suites with sandbox factory/setup updates.

  • SDK clients:
    • JS (js/src/sandbox.ts): Send E2B-Traffic-Access-Token (when present) with runCode and context APIs (/contexts, /contexts/:id, /contexts/:id/restart); standardize headers.
    • Python (code_interpreter_async.py, code_interpreter_sync.py): Include Content-Type and E2B-Traffic-Access-Token (plus existing X-Access-Token) for execute and all context endpoints; minor header construction cleanup.
  • Tests & setup:
    • JS tests: Add secureSandboxTest (secure mode, network.allowPublicTraffic=false) and corresponding secure variants of basic/context tests; enhance tests/setup.ts with sandboxTestId metadata and configurable sandboxOpts.
    • Python tests: Add secure-traffic tests for async/sync basic and context flows using new factories; refactor conftest.py to provide sandbox_factory, async_sandbox_factory, persistent event loop, and per-test sandbox_test_id metadata.
  • Changeset: Patch releases for @e2b/code-interpreter and @e2b/code-interpreter-python.

Written by Cursor Bugbot for commit 160b4fc. This will update automatically on new commits. Configure here.

@chatgpt-codex-connector
Copy link

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@mishushakov mishushakov enabled auto-merge (squash) November 26, 2025 15:33
@jakubno jakubno disabled auto-merge November 26, 2025 17:59
Copy link
Member

@jakubno jakubno left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One small NIT otherwise lgtm

@mishushakov mishushakov enabled auto-merge (squash) November 26, 2025 18:07
@mishushakov mishushakov merged commit 59ec6f4 into main Nov 26, 2025
9 checks passed
@mishushakov mishushakov deleted the secure-traffic-access branch November 26, 2025 18:10
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.

3 participants