Skip to content

feat: support long polling requests and event-source responses#538

Closed
ronag wants to merge 3 commits intomainfrom
long-polling-event-source
Closed

feat: support long polling requests and event-source responses#538
ronag wants to merge 3 commits intomainfrom
long-polling-event-source

Conversation

@ronag
Copy link
Copy Markdown
Member

@ronag ronag commented Feb 5, 2021

No description provided.

@ronag ronag added the Status: help-wanted This issue/pr is open for contributions label Feb 5, 2021
@ronag
Copy link
Copy Markdown
Member Author

ronag commented Feb 5, 2021

Needs tests

@ronag ronag force-pushed the long-polling-event-source branch from 6e79ed0 to a455b52 Compare February 5, 2021 12:35
@ronag ronag closed this Feb 5, 2021
@ronag ronag reopened this Feb 5, 2021
@ronag ronag force-pushed the long-polling-event-source branch from a455b52 to 020f6ea Compare February 5, 2021 12:39
@ronag
Copy link
Copy Markdown
Member Author

ronag commented Feb 5, 2021

This logic is only relevant when pipelining > 1 and ensures that no more requests are enqueued.

@codecov-io
Copy link
Copy Markdown

codecov-io commented Feb 5, 2021

Codecov Report

Merging #538 (020f6ea) into master (8aa5ad1) will decrease coverage by 0.07%.
The diff coverage is 93.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #538      +/-   ##
==========================================
- Coverage   99.63%   99.56%   -0.08%     
==========================================
  Files          16       16              
  Lines        1365     1375      +10     
==========================================
+ Hits         1360     1369       +9     
- Misses          5        6       +1     
Impacted Files Coverage Δ
lib/core/symbols.js 100.00% <ø> (ø)
lib/core/client.js 99.30% <92.85%> (-0.17%) ⬇️
lib/core/request.js 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8aa5ad1...e88b0f7. Read the comment docs.

@ronag ronag force-pushed the long-polling-event-source branch from db70669 to e88b0f7 Compare February 5, 2021 19:17
Base automatically changed from master to main March 12, 2021 21:33
@ronag ronag force-pushed the main branch 3 times, most recently from 87a911d to f5ec39e Compare April 8, 2021 22:46
@ronag ronag added the enhancement New feature or request label May 31, 2021
@ronag ronag force-pushed the main branch 2 times, most recently from becd967 to b33dffa Compare June 7, 2021 20:38
@ronag ronag force-pushed the main branch 2 times, most recently from 6679898 to 4b5ed0b Compare July 19, 2021 07:13
@ronag ronag force-pushed the main branch 8 times, most recently from e4db54a to a3994a7 Compare August 14, 2021 19:13
@ronag ronag force-pushed the main branch 6 times, most recently from 5d7e3e4 to d684ce0 Compare August 18, 2021 17:27
@ronag ronag force-pushed the main branch 2 times, most recently from a2ad318 to 904e045 Compare October 29, 2021 12:45
@ronag ronag force-pushed the main branch 3 times, most recently from dcb1656 to af20fdc Compare November 22, 2021 20:15
@ronag ronag force-pushed the main branch 2 times, most recently from 63eab1d to 47eb207 Compare December 12, 2021 12:27
@metcoder95
Copy link
Copy Markdown
Member

This is an implementation of SSR or pure long-polling? (I'm asking because I see the text/event-stream header involved) :)

Copy link
Copy Markdown
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

wow!

@ronag ronag closed this Feb 25, 2024
@Uzlopak Uzlopak deleted the long-polling-event-source branch April 7, 2024 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Status: help-wanted This issue/pr is open for contributions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants