[Home]

Summary:ASTERISK-01441: [patch] Uniqueids missing in Link and Unlink manager events
Reporter:mjohnston (mjohnston)Labels:
Date Opened:2004-04-20 12:53:58Date Closed:2004-09-25 02:51:24
Priority:TrivialRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) link-uniqueid.patch
Description:The manager Link and Unlink events give channel names, but not Uniqueids.  This means a lot of duplicated effort if a manager app wants to track calls based on uniqueid instead of channel, since it has to keep a database of channel vs. uniqueid in case of Link or Unlink.  This trivial patch adds Uniqueid1 and Uniqueid2 (analogous to Channel1 and Channel2) to Link and Unlink.  This change shouldn't break any well-written manager applications, since all existing fields are left intact and as-is.

****** ADDITIONAL INFORMATION ******

I'm not sure this is the only place that should be reporting Uniqueid and isn't, but it's the only one I've noticed.  A disclaimer should be sitting on the fax at Digium now.
Comments:By: Mark Spencer (markster) 2004-04-20 19:24:27

This patch does not apply.  Are you sure this is binary accurate to your cvs diff -u and didn't touch a windows box or get cut/pasted somehow?

By: James Golovich (jamesgolovich) 2004-04-21 00:48:18

I manually applied the patch and commited it to CVS HEAD.

Thank you very much.  This will be very useful