Skip to content

Commit d88ccd9

Browse files
fix: use single newline for SSE retry field to avoid event dispatch
Address review feedback: sending `retry: N\n\n` could trigger SSE event dispatch behavior in some client implementations. Using a single `\n` ensures the retry field is processed without potentially dispatching an empty event.
1 parent aa02bb7 commit d88ccd9

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

src/server/streamableHttp.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -815,8 +815,9 @@ export class StreamableHTTPServerTransport implements Transport {
815815
const stream = this._streamMapping.get(streamId);
816816
if (stream) {
817817
// If a custom retry interval is provided, send it before closing
818+
// Use single \n (not \n\n) to avoid triggering SSE event dispatch
818819
if (retryInterval !== undefined) {
819-
stream.write(`retry: ${retryInterval}\n\n`);
820+
stream.write(`retry: ${retryInterval}\n`);
820821
}
821822
stream.end();
822823
this._streamMapping.delete(streamId);

0 commit comments

Comments
 (0)