Repository navigation
fix(code-interpreter): forward context authentication tokens - #1955
devin-ai-integration[bot] wants to merge 1 commit into
Conversation
Co-Authored-By: mish@e2b.dev <mish@e2b.dev>
|
I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".
|
There was a problem hiding this comment.
TASTE.md review: no violations found. The change forwards X-Access-Token in all four context operations in JS, sync Python, and async Python alike (T-1/T-2 parity), and keeps the token on the internal envdAccessToken / _envd_access_token field (T-18). No public API, options, errors, or docstrings change.
Package ArtifactsBuilt from 824637a. Download artifacts from this workflow run. JS SDK ( npm install ./e2b-2.53.2-devin-1791368952-interpreter-auth.0.tgzCLI ( npm install ./e2b-cli-2.21.2-devin-1791368952-interpreter-auth.0.tgzCode Interpreter JS SDK ( npm install ./e2b-code-interpreter-2.8.3-devin-1791368952-interpreter-auth.0.tgzDesktop JS SDK ( npm install ./e2b-desktop-2.4.1-devin-1791368952-interpreter-auth.0.tgzPython SDK ( pip install ./e2b-2.53.1+devin.1791368952.interpreter.auth-py3-none-any.whlCode Interpreter Python SDK ( pip install ./e2b_code_interpreter-2.10.3+devin.1791368952.interpreter.auth-py3-none-any.whlDesktop Python SDK ( pip install ./e2b_desktop-2.6.1+devin.1791368952.interpreter.auth-py3-none-any.whl |
|
The Cloudflare CI leg failed with See the Cloudflare job (112754571531) in https://github.com/e2b-dev/E2B/pull/1955/checks. |
Summary
Forward
X-Access-Tokenfor context creation, listing, restart, and removal in JS and both Python clients, preserving traffic-token routing and tokenless clients. This lets the same sandbox credential authenticate execution and context management when the interpreter template enables authentication. Ship these SDK releases before the token-only e2b-dev/code-interpreter#342; envd provisioning is https://github.com/e2b-dev/belt/pull/4368. Jupyter isolation in e2b-dev/code-interpreter#343 is independent and needs no SDK update.Usage remains unchanged:
const context = await sandbox.createCodeContext(); await sandbox.restartCodeContext(context)(Python:context = sandbox.create_code_context(); sandbox.restart_code_context(context)).Related: https://linear.app/e2b/issue/SDK-363/verify-code-interpreter-context-endpoint-auth-requirements
Verification: 17 focused JS and 18 Python tests passed; format, lint and typecheck passed. The new auth tests fail with HTTP 401 on the pre-fix clients and pass with token forwarding restored.
Link to Devin session: https://app.devin.ai/sessions/23861bced08343e59c14adf841498b12
Open in Devin Desktop: https://app.devin.ai/desktop/session/23861bced08343e59c14adf841498b12?variant=devin
Requested by: @mishushakov