nodejs/undici

How to handle text/event-stream?

Closed

#536 opened on Feb 4, 2021

View on GitHub
 (4 comments) (1 reaction) (0 assignees)JavaScript (5,319 stars) (444 forks)batch import
Docseventsourcegood first issue

Description

I think text/event-stream responses should be handled as an upgrade and the socket released from the undici client (like with websockets).

Edit: this is done but docs for EventSource are missing.

Contributor guide

Tech stack
nodejs
Domain
documentation
Issue type
documentation
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
1
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
mostly clear
Prerequisites
None
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
80
Research direction
The issue states that the implementation for handling text/event stream is complete but documentation for EventSource is missing. First, review the repository's documentation structure (e.g., docs/ directory) to understand the style. Then, examine the existing code related to EventSource or event stream handling to document the API usage. Specific files to look at include any existing doc files and the source code for EventSource. Finally, open a PR adding the missing documentation.