-
Notifications
You must be signed in to change notification settings - Fork 591
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
Consider removing the web-hook spec #476
Comments
To clarify, I hope this is both better for the CloudEvents spec, because the goal of "no processing model" is much clearer if we don't deal with this anywhere; and better for the webhook spec because it can dive into this topic and can be opinionated when it needs to be. |
The spec really is a required profile for achieving ad-hoc event delivery interop over a protocol with a million knobs like HTTP, IMO. It defines the HTTP method, it narrows down the auth options, and provides a simple DoS abuse defense model. I think it'd be great for the spec to be moved into a more appropriate forum, but I do think we need it and CloudEvents implementations ought to implement it. Until we have a better home (IETF?), it should best stay here. The alternative were to pull the spec back into the HTTP binding. |
I've started a discussion with some of our (Microsoft) standards folks about a better venue for this spec. In that email thread I wrote this:
I actually think it'd be a huge loss if we cut that last hop spec out and moved it elsewhere. |
I'm going to close this since I can't think of a better place for it right now and we can always choose to move it later. |
See: #414 (comment)
As @cneijenhuis points out, this doc (while useful) isn't really related to CloudEvents and as such might be better off living some place else. Additionally, for the most part CloudEvents is trying to stay away from defining the processing model for consumers (or even processing model for transports), and this spec gets into those topics.
/cc @clemensv
The text was updated successfully, but these errors were encountered: