Skip to content

feat(sse): make parser limits configurable via SSEParserConfig - #1149

Open
ehsavoie wants to merge 1 commit into
a2aproject:mainfrom
ehsavoie:issue_1123
Open

feat(sse): make parser limits configurable via SSEParserConfig#1149
ehsavoie wants to merge 1 commit into
a2aproject:mainfrom
ehsavoie:issue_1123

Conversation

@ehsavoie

@ehsavoie ehsavoie commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator
  • Default max line length raised from 64 KB to 1 MB to fix interop with servers emitting large single data: lines
  • Limits configurable via a2a.sse.* properties (MicroProfile Config) or programmatically through SSEParserConfig builder

Fixes #1123 🦕

@ehsavoie
ehsavoie requested a review from kabir September 9, 2026 14:40
@ehsavoie ehsavoie mentioned this pull request Sep 9, 2026
Closed
kabir
kabir reviewed Sep 10, 2026
View reviewed changes
Comment thread server-common/src/main/resources/META-INF/a2a-defaults.properties Outdated
@ehsavoie
feat(sse): add SSEParserConfig, raise default per-line limit to 1 MB
c0569cb 
Fixes a2aproject#1123

Signed-off-by: Emmanuel Hugonnet <ehugonne@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kabir kabir kabir left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Bug]: SSE client rejects any event whose data arrives on a single line over 64KB, with no way to configure the limit

2 participants

@ehsavoie @kabir