<p>George Joseph <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.asterisk.org/10311">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">app_confbridge:  Use bridge join hook to send join and leave events<br><br>The first attempt at publishing confbridge events to participants<br>involved publishing them at the same time stasis events were<br>created.  This caused issues with bridge and channel locks.  The<br>second attempt involved publishing them when the stasis events<br>were received by the code that published the confbridge AMI events.<br>This caused timing issues because, depending on resources available,<br>the event could be received before channels actually joined the<br>bridge and would therefore fail to send messages to the participant.<br><br>This attempt reverts to the original mechanism with one exception.<br>The join and leave events are published via bridge join and leave<br>hooks.  This guarantees the states of the channels and bridge and<br>provides deterministic timing for event publishing.<br><br>Change-Id: I2660074f8a30a5224cb953d5e047ee84484a9036<br>---<br>M apps/app_confbridge.c<br>M apps/confbridge/confbridge_manager.c<br>M apps/confbridge/include/confbridge.h<br>3 files changed, 75 insertions(+), 14 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/11/10311/2</pre><p>To view, visit <a href="https://gerrit.asterisk.org/10311">change 10311</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/10311"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 16 </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: I2660074f8a30a5224cb953d5e047ee84484a9036 </div>
<div style="display:none"> Gerrit-Change-Number: 10311 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 (1000185) </div>