Skip to content

fix: support HTTPS for live-stream chunk delivery#80

Merged
weiesky merged 1 commit intoweiesky:mainfrom
Majorshi:main
May 2, 2026
Merged

fix: support HTTPS for live-stream chunk delivery#80
weiesky merged 1 commit intoweiesky:mainfrom
Majorshi:main

Conversation

@Majorshi
Copy link
Copy Markdown
Contributor

@Majorshi Majorshi commented May 2, 2026

sendStreamChunk used http.request unconditionally, which silently fails when the server listens on HTTPS (self-signed cert). This breaks the onStreamChunk plugin hook — no content_delta SSE events are broadcast during streaming.

  • Add https import and _liveProtocol state to interceptor
  • setLivePort now accepts optional protocol parameter
  • sendStreamChunk selects http/https module based on _liveProtocol
  • server.js passes serverProtocol to setLivePort on listen

sendStreamChunk used http.request unconditionally, which silently
fails when the server listens on HTTPS (self-signed cert). This
breaks the onStreamChunk plugin hook — no content_delta SSE events
are broadcast during streaming.

- Add https import and _liveProtocol state to interceptor
- setLivePort now accepts optional protocol parameter
- sendStreamChunk selects http/https module based on _liveProtocol
- server.js passes serverProtocol to setLivePort on listen

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@weiesky weiesky merged commit adae203 into weiesky:main May 2, 2026
1 check passed
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