Skip to content

RPC transport include default headers - #765

Closed
mishushakov wants to merge 9 commits into
mainfrom
envd-rpc-inlude-default-headers
Closed

RPC transport include default headers#765
mishushakov wants to merge 9 commits into
mainfrom
envd-rpc-inlude-default-headers

Conversation

@mishushakov

@mishushakov mishushakov commented Jun 7, 2025

Copy link
Copy Markdown
Member
  • Modified rpc transport to send default headers like user agent, client version, etc.
  • We want this to be able to change backend logic in a non-breaking way for older clients

Browser tests failing due to e2b-dev/infra#744

@mishushakov mishushakov self-assigned this Jun 7, 2025
@mishushakov mishushakov added the Improvement Improvement for current functionality label Jun 7, 2025
@changeset-bot

changeset-bot Bot commented Jun 7, 2025

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: a2a7823

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@e2b/python-sdk Patch
e2b Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@mishushakov mishushakov added sdk Improvements or additions to SDKs and removed Improvement Improvement for current functionality labels Jun 7, 2025
Comment thread packages/js-sdk/src/sandbox/index.ts Outdated
@jakubno

jakubno commented Jul 9, 2025

Copy link
Copy Markdown
Member

Let's wait on e2b-dev/infra#847 and e2b-dev/infra#745 and decide what headers we want to set/update based on that

@jakubno

jakubno commented Jul 28, 2025

Copy link
Copy Markdown
Member

We are sending unique User-Agent(#830) for each version and SDK now, so this isn't needed at the end, closing

@jakubno jakubno closed this Jul 28, 2025
@jakubno
jakubno deleted the envd-rpc-inlude-default-headers branch July 28, 2025 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sdk Improvements or additions to SDKs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants