<p style="white-space: pre-wrap; word-wrap: break-word;">After thinking this through more and looking at the code I don't think this is the correct solution. The only case where you can have a subscription subscribed multiple times is actually using the forwarding code. You can't directly do it. The intention of the forwarding code is to ensure that messages for one topic are forwarded to the subscribers of another. The behavior right now results in multiple messages, which is not ideal. I'm worried however that with this change once the first forward is canceled the subsequent ones are no longer in effect, so messages won't be forwarded. This goes against the intention of the forwarding.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I think there's two options:<br>1. As long as 1 forwarding is still active the forwarding still occurs. It is only when the last one is canceled that it stops.<br>2. Trying to set up forwarding again involving the same topics results in an error that they are already forwarded.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not certain which is the best option really. On one hand with option 1 it would just work. On the other hand trying to forward between the same topics again seems like a bug.</p><p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4; color: #000000;">Code-Review -1</span></p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/11201">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/11201">change 11201</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/11201"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I00bd627e4cc21aba0535d2695460a04f06f6e699 </div>
<div style="display:none"> Gerrit-Change-Number: 11201 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: sungtae kim <pchero21@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: sungtae kim <pchero21@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 05 Apr 2019 11:19:43 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>