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

Events are loosely constrained #42

Closed
egekorkan opened this issue Oct 5, 2020 · 2 comments
Closed

Events are loosely constrained #42

egekorkan opened this issue Oct 5, 2020 · 2 comments
Labels
FPWD Feedback feedback on FPWD-ready documents

Comments

@egekorkan
Copy link
Contributor

egekorkan commented Oct 5, 2020

  • The events section is way too loose to allow out-of-the-box-interop (OOTBI). With WS in there, OOTBI is impossible, similar-ish with WebHooks. No experience at all with both of them from the plugfests. If this spec is not bringing OOTBI, its main argument to exist becomes less obvious.
  • Any restriction in subprotocols?

Non-optimal solution: Allow only longpoll for events in HTTP.

@egekorkan egekorkan added the FPWD Feedback feedback on FPWD-ready documents label Oct 5, 2020
@mlagally
Copy link
Contributor

mlagally commented Oct 7, 2020

Discussion in arch call on 7.10.:
Use Longpoll for now, since we have experience with it, explore SSE at a later time.

For performance we may want to use websockets at a later point in time in a future spec.

@relu91
Copy link
Member

relu91 commented Oct 13, 2020

Also, the spec should refer to them as subprotocols:

The supported protocols for events are WebHooks, WebSockets, SSE and Long polling.

The supported subprotocols for HTTP(S) are: WebHooks, SSE, and Long polling

mlagally added a commit that referenced this issue Sep 16, 2021
Define protocol binding for events - closes #42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FPWD Feedback feedback on FPWD-ready documents
Projects
None yet
Development

No branches or pull requests

3 participants