Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Generic ServerSentEvents (#2952) #2985

Merged
merged 1 commit into from
Aug 2, 2024
Merged

Generic ServerSentEvents (#2952) #2985

merged 1 commit into from
Aug 2, 2024

Conversation

987Nabil
Copy link
Contributor

fixes #2952
/claim #2952

@987Nabil 987Nabil requested review from jdegoes and vigoo as code owners July 30, 2024 22:10
@algora-pbc algora-pbc bot mentioned this pull request Jul 30, 2024
@987Nabil 987Nabil force-pushed the generic-sse branch 2 times, most recently from 480d329 to 4f21ce0 Compare July 30, 2024 22:26
ZPipeline.chunks[Byte].map(_.asString) >>> ZPipeline
.splitOn("\n\n")
.mapZIO(s => {
val lines = s.linesIterator
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This logic for encoding and decoding should have some tests associated with it. 👍

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jdegoes This is actually more something for #2953 since receiving SSE is not yet impl. I have a draft PR open for that #2987 that I am working on right now. Are you okay if I add the tests there?

@jdegoes jdegoes merged commit 66d3eea into zio:main Aug 2, 2024
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Generic ServerSentEvents
2 participants