-
Notifications
You must be signed in to change notification settings - Fork 708
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
Remove clear_multicast_endpoints() in unsubscribe. #651
base: master
Are you sure you want to change the base?
Conversation
The clearing multicast endpoints stops any multicast messages receiving without recovering if TTLs are not expiring. Co-authored-by: Aram Khzarjyan <[email protected]> Signed-off-by: Gunnar Andersson <[email protected]>
PR looks good to me. |
Hi @akhzarj could you give an example of how we can reproduce this issue? |
if one wants to unsubscribe to a set of service/instance, why would you want to receive any more messages from that specific endpoint? |
Yes in your case that true. |
@akhzarj i am trying to reproduce one your issue. i am using the attached config files.
thanks |
Hi @duartenfonseca |
@akhzarj do you have any pcap logs of the behaviour? because even with the TTL 0xFFFFF it does not happen. When using the subscribe-sample/notify-sample, the new offer always restores the endpoint so it does not stay closed. |
on the logs we still see the messages being received on the client, for the other event which was not unsubscribed |
Hi @duartenfonseca |
@akhzarj an example would be helpful, thank you! |
Author: Aram Khzarjyan [email protected], MBition GmbH.
Remove clear_multicast_endpoints() in unsubscribe. #651
The clearing multicast endpoints stops any multicast messages
receiving without recovering if TTLs are not expiring.
The program was tested solely for our own use cases, which might differ from yours.
The submission is provided under the main project license (LICENSE file in root of project).
Provider Information