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

DSUBm_005: Should $events operation be required #13

Open
scontento opened this issue Oct 30, 2023 · 3 comments
Open

DSUBm_005: Should $events operation be required #13

scontento opened this issue Oct 30, 2023 · 3 comments
Labels
open issue Issues which we invite comments

Comments

@scontento
Copy link
Contributor

scontento commented Oct 30, 2023

(see DSUBm_005 issues.html)
The Resource Subscription Search [ITI-113] define the $events operation that permits to retrieve from the Resource Notification Broker the events associated to a Subscription. This operation could be use to retrieve puntually the missed event when, for example, a connection problem occur and the Resource Notification Recipient does note receive the notification. But, this operation requires the Resource Notification Broker to be capable of maintain previous events associated with the Subscriptions. Should the supporting of this operation be required for the Resource Notification Broker or should be maintain optional?

@JohnMoehrke JohnMoehrke changed the title DSUBm_005 DSUBm_005: Should $events operation be required Nov 17, 2023
@RicardoQuintano
Copy link

I think it is important to set the $events operation as REQUIRED.

The fact of a Recipient NOT receiving a notification (e.g., due to connection problems) may result in a Task not done by the clinical staff that might reflect in bad consequences for the patient (e.g., not getting a medication because the physician was not notified that lab tests where ready).

@GinoCanessa
Copy link

I agree that $events needs to be required if the channel is not "reliable" (guaranteed delivery). Given that DSUBm prefers rest-hook, there needs to be some way for clients to recover in the case of errors.

@scontento
Copy link
Contributor Author

Leave this as a open issue for discussion.

Leaving the operation as optional let this operation is still available if implemented. (maybe a reccomendation could be useful to the reader/implementers?)

Considering the requirement of this operation implies that the Broker shall keep track of the history of the events that have occurred.
Is this consideration relevant to the decision on this operation?

@scontento scontento added the open issue Issues which we invite comments label Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
open issue Issues which we invite comments
Projects
None yet
Development

No branches or pull requests

3 participants