-
Notifications
You must be signed in to change notification settings - Fork 31
Clarifications for asynchronous responses #575
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
base: main
Are you sure you want to change the base?
Clarifications for asynchronous responses #575
Conversation
|
|
||
| An asynchronous response is not really an event. An event is something that can happen or not, once or several times and it is the occurrence of the event that carries the main information, whereas an asynchronous response represents the metainformation carried over in the same format as it would be a synchronous response and will happen once and only once. | ||
|
|
||
| This is the rationale to NOT use [CAMARA cloudevents-based model for event notification](./CAMARA-API-Event-Subscription-and-Notification-Guide.md#3-event-notification) for these kind of scenarios but to use the same response model as in an analogous synchronous scenario. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'same response model' -> may be worth stating explicitly that an asychronous response would typically involve a different response status code (202 Acceptedfor the non-error scenarios )
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Making some updates. Also considering offline proposal by @rartych
| $ref: "#/components/responses/<schema-name>" | ||
| ``` | ||
|
|
||
| ##### 5.7.6.1 Asynchronous responses |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have 5.7.6.1 section proposed in #567
We need to decide on the sequence and resolve the conflict.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tentatively i assign 5.7.6.2 in this PR
patrice-conil
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
What type of PR is this?
What this PR does / why we need it:
This PR address some guidelines with regards to scenarios where API response is provided asynchronously. It is stated that the response model does NOT have to follow CAMARA cloudevents-based event notification model as the conclusion from Issue #533.
Which issue(s) this PR fixes:
Fixes #533
Does this PR introduce a breaking change?
Special notes for reviewers:
Changelog input
Additional documentation
This section can be blank.