<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://reviewboard.asterisk.org/r/2690/">https://reviewboard.asterisk.org/r/2690/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Ship It!</pre>
<br />
<p>- rmudgett</p>
<br />
<p>On July 22nd, 2013, 3:05 p.m. UTC, Matt Jordan wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/static/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers.</div>
<div>By Matt Jordan.</div>
<p style="color: grey;"><i>Updated July 22, 2013, 3:05 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://issues.asterisk.org/jira/browse/ASTERISK-21699">ASTERISK-21699</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
Asterisk
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">In previous versions of Asterisk, when masquerades happened a lot and external systems needed to track and maintain the state of channels during a masquerade, the rename events that showed which channel had been replaced by a zombie made sense. (Or at least, could be justified).
Now, masquerades are very rare and are hidden. Their last remaining vestige of life (for some definition of "life") is the renaming of the channel to "%s<ZOMBIE>".
And of course, this still causes problems.
When a call pickup occurs (one of the only two scenarios where a masquerade still occurs), the channel that was replaced in the pickup gets zombiefied. When that occurs, CDRs will start to freak out a bit, as they start to receive updates for channels that they can't identify. I would imagine external systems monitoring CEL and/or AMI will also get a bit confused.
While we could track rename events, masquerade callbacks, etc. - it all is rather meaningless now. When a masquerade occurs today, it only occurs when a channel is replaced and the replaced channel is immediately hung up. There's no need for the rename.
This kills the rename (and as a result, makes call pickup in CDRs work. Yay!)</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/trunk/main/channel.c <span style="color: grey">(395072)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/2690/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>