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

issue 7571 - SSE - add Keep-Alive Messages definition #7572

Closed
wants to merge 2 commits into from

Conversation

liran2000
Copy link

@liran2000 liran2000 commented Feb 3, 2022

Issue #7571: Server-Sent Events (SSE) - add Keep-Alive Messages definition

"...authors can include a comment line (one starting with a ':' character) every 15 seconds or so."

This is helpful for SSE clients. The SSE client can read the keep-alive messages, and detect whether the connection is alive.

Thing is, since the keep-alive messages mechanism is not mandatory, the SSE client can use this condition only if the keep-alive mechanism is used, since when it is not used, it is a valid scenario that no message is received for a long time, even when the stream is alive.

Problem:
keep-alive message is not well-defined.
Suggested solution - added the part in bold:
authors can include a comment line (one starting with a ':' character) starting with ":keepalive" every 15 seconds or so.

This way, the SSE client can detect if keep-alive mechanism is used by checking if it received a keep-alive message as defined.


/acknowledgements.html ( diff )
/browsers.html ( diff )
/browsing-the-web.html ( diff )
/canvas.html ( diff )
/custom-elements.html ( diff )
/dom.html ( diff )
/form-control-infrastructure.html ( diff )
/grouping-content.html ( diff )
/history.html ( diff )
/iframe-embed-object.html ( diff )
/images.html ( diff )
/index.html ( diff )
/input.html ( diff )
/interaction.html ( diff )
/origin.html ( diff )
/references.html ( diff )
/scripting.html ( diff )
/sections.html ( diff )
/semantics.html ( diff )
/server-sent-events.html ( diff )
/structured-data.html ( diff )
/syntax.html ( diff )
/text-level-semantics.html ( diff )
/urls-and-fetching.html ( diff )
/web-messaging.html ( diff )
/webappapis.html ( diff )
/window-object.html ( diff )
/worklets.html ( diff )

@liran2000 liran2000 closed this Feb 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant