Skip to content

fix: prevent SSE voucher posts from consuming charges#561

Merged
brendanjryan merged 1 commit into
mainfrom
brendanjryan/fix-sse-session-example
Jun 17, 2026
Merged

fix: prevent SSE voucher posts from consuming charges#561
brendanjryan merged 1 commit into
mainfrom
brendanjryan/fix-sse-session-example

Conversation

@brendanjryan

Copy link
Copy Markdown
Collaborator

Summary

  • Prevented SSE session voucher management POSTs from being counted as billable content charges.
  • Updated the SSE session example to use explicit per-token stream charges with the TIP-1034 session flow.
  • Added a patch changeset for the session accounting fix.

Motivation

SSE voucher updates are management traffic. Counting them as content charges consumed voucher headroom before the stream could commit its reserved charge, which could terminate the stream.

Key design considerations

  • Preserved non-SSE POST content accounting.
  • Kept SSE content precharge behavior for GET streams.
  • Removed default fee sponsorship from the example so the funded client pays channel-open gas.

@pkg-pr-new

pkg-pr-new Bot commented Jun 17, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/mppx@561

commit: a44475d

@brendanjryan brendanjryan marked this pull request as ready for review June 17, 2026 14:47
@brendanjryan brendanjryan merged commit 5e27ef2 into main Jun 17, 2026
16 of 22 checks passed
@brendanjryan brendanjryan deleted the brendanjryan/fix-sse-session-example branch June 17, 2026 15:06
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.

1 participant