Skip to content

feat: add sandbox endpoint auth headers#492

Merged
hittyt merged 3 commits intoalibaba:mainfrom
ninan-nn:feature/endpoint_headers
Mar 20, 2026
Merged

feat: add sandbox endpoint auth headers#492
hittyt merged 3 commits intoalibaba:mainfrom
ninan-nn:feature/endpoint_headers

Conversation

@ninan-nn
Copy link
Collaborator

@ninan-nn ninan-nn commented Mar 19, 2026

Summary

  • add per-sandbox egress auth header generation and propagation through lifecycle endpoint responses
  • wire Docker and Kubernetes runtimes to persist the auth token in runtime metadata, inject it into the egress sidecar, and merge it with existing endpoint headers
  • update SDK layers to consume endpoint headers for execd/egress access, including server-proxy flows
  • fix Docker server-proxy endpoint resolution for bridge sandboxes with egress sidecar by falling back to host-mapped endpoint resolution when internal IP resolution is not applicable
  • add unit and multi-language e2e coverage for direct and server-proxy egress access

Testing

  • Not run (please update if local/CI verification has already been done)
  • Unit tests added/updated in this PR
  • Integration tests
  • e2e / manual verification coverage added/updated in this PR

Breaking Changes

  • None
  • Yes (describe impact and migration path)

Checklist

  • Linked Issue or clearly described motivation
  • Added/updated docs (if needed)
  • Added/updated tests (if needed)
  • Security impact considered
  • Backward compatibility considered

@ninan-nn ninan-nn force-pushed the feature/endpoint_headers branch from d818a8b to 6d47025 Compare March 20, 2026 02:33
@ninan-nn ninan-nn changed the title add egress auth headers egress auth headers Mar 20, 2026
@ninan-nn ninan-nn changed the title egress auth headers feat: add sandbox endpoint auth headers Mar 20, 2026
@ninan-nn ninan-nn marked this pull request as ready for review March 20, 2026 02:40
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6d47025464

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Copy link
Collaborator

@Pangjiping Pangjiping left a comment

Choose a reason for hiding this comment

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

LGTM

@Pangjiping Pangjiping added feature New feature or request bug Something isn't working labels Mar 20, 2026
Copy link
Collaborator

@hittyt hittyt left a comment

Choose a reason for hiding this comment

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

LGTM

@hittyt hittyt merged commit 0647691 into alibaba:main Mar 20, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working component/server feature New feature or request sdks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants