Skip to content

use header for session ids in streamable http GET streams#472

Merged
deathbyknowledge merged 2 commits intomainfrom
use-header-for-session-id
Sep 15, 2025
Merged

use header for session ids in streamable http GET streams#472
deathbyknowledge merged 2 commits intomainfrom
use-header-for-session-id

Conversation

@deathbyknowledge
Copy link
Copy Markdown
Member

In the Streamable HTTP handler we were reading the session ID from query parameters instead of the proper header.

That was the case for SSE but not for Streamable HTTP.

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Sep 15, 2025

🦋 Changeset detected

Latest commit: 0fc912a

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

This PR includes changesets to release 1 package
Name Type
agents 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

@deathbyknowledge deathbyknowledge marked this pull request as ready for review September 15, 2025 18:48
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Sep 15, 2025

Open in StackBlitz

npm i https://pkg.pr.new/cloudflare/agents@472

commit: 0fc912a

@deathbyknowledge deathbyknowledge merged commit 2d0d2e1 into main Sep 15, 2025
3 checks passed
@deathbyknowledge deathbyknowledge deleted the use-header-for-session-id branch September 15, 2025 18:52
@threepointone threepointone mentioned this pull request Sep 15, 2025
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.

2 participants