Skip to content
This repository has been archived by the owner on Oct 20, 2022. It is now read-only.

cep: clean subscription #44

Merged
merged 2 commits into from
Feb 8, 2016
Merged

cep: clean subscription #44

merged 2 commits into from
Feb 8, 2016

Conversation

PascaleBorscia
Copy link
Member

Fixes issue #40

The CEP subscribes to the Broker entities configured in the configuration file.
The CEP does not persist subscriptions.
If the CEP is restarted (after a crash or a user stop), the CEP done again the subscriptions.
The broker sends two notifications (during 1H of subscription time), and one of them is invalid and therefore rejected.

It is proposed to reject the notification but also to send the Broker an unregistration so the Broker could clean the subscription. The Broker will send a single notification.

The unsubscribe will be made only if the subscription is invalid and if the susbcription third mechanism is not engaged.

- send unsubscription to the originator when the subscriptionId is invalid
- move the unsubscribe code from NgsiController to SubscriptionManager
@PascaleBorscia PascaleBorscia self-assigned this Feb 8, 2016
@PascaleBorscia PascaleBorscia added this to the 0.1.5 milestone Feb 8, 2016
@marcc-orange marcc-orange changed the title Clean subscription cep: clean subscription Feb 8, 2016
@marcc-orange
Copy link
Contributor

LGTM 👍

marcc-orange added a commit that referenced this pull request Feb 8, 2016
@marcc-orange marcc-orange merged commit 54a6f23 into master Feb 8, 2016
@marcc-orange marcc-orange deleted the clean-subscription branch February 8, 2016 13:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants