Skip to content

Update the time for each request#1283

Merged
jakubno merged 1 commit into
mainfrom
fix-sandbox-sent-to-envd
Oct 1, 2025
Merged

Update the time for each request#1283
jakubno merged 1 commit into
mainfrom
fix-sandbox-sent-to-envd

Conversation

@jakubno
Copy link
Copy Markdown
Member

@jakubno jakubno commented Oct 1, 2025

Note

Centralizes envd init request body construction in the retry function, adding a per-attempt timestamp and updating the call site accordingly.

  • Orchestrator Sandbox (packages/orchestrator/internal/sandbox/envd.go):
    • doRequestWithInfiniteRetries: Refactored signature to accept envVars and hyperloopIP; constructs JSON body internally with envVars, hyperloopIP, accessToken, and a per-attempt timestamp.
    • initEnvd: Updated to use the new API (no local JSON marshalling); uses http.MethodPost and passes envVars and hyperloopIP to the retry function.

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

@jakubno jakubno added the improvement Improvement for current functionality label Oct 1, 2025
@e2b-request-same-site-reviewers e2b-request-same-site-reviewers Bot requested review from sitole and removed request for ValentaTomas and dobrac October 1, 2025 20:05
@jakubno jakubno removed the request for review from sitole October 1, 2025 20:05
@jakubno jakubno enabled auto-merge (squash) October 1, 2025 20:08
@jakubno jakubno merged commit 984e580 into main Oct 1, 2025
25 of 26 checks passed
@jakubno jakubno deleted the fix-sandbox-sent-to-envd branch October 1, 2025 20:16
ValentaTomas pushed a commit that referenced this pull request May 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improvement for current functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants