Ticket #3684 (closed Bugs: duplicate)

Opened 3 years ago

Last modified 2 years ago

Recordings from other extensions show up in ARI Call Monitor

Reported by: ip-rob Assigned to:
Priority: minor Milestone: 2.6
Component: User Portal Version: 2.5-branch
Keywords: recordings, call monitor Cc:
Confirmation: Need Feedback SVN Revision (if applicable):
Backend Engine: Asterisk 1.4.x Backend Engine Version:

Description

The matching criteria in the ARI portal is not sufficient to properly filter recording files. There are several cases where recordings from other extensions will show up if the calls were made about the same time even if aggressive matching is turned off.

I've created a patch which adds in the requirement that the extension, outboundCID, or unique ID must be in the filename unless aggressive matching is enabled. Aggressive matching works the same way.

The result is that some matches which might have shown up before will not unless asterisk-addons are compiled with the uniqueID option. However, the patch will prevent recordings from wrong extensions from showing up.

Attachments

callmonitor.module.patch (1.5 kB) - added by ip-rob on 05/19/09 21:40:38.
Patch for callmonitor.module file in recordings/modules for user portal

Change History

05/19/09 21:40:38 changed by ip-rob

  • attachment callmonitor.module.patch added.

Patch for callmonitor.module file in recordings/modules for user portal

05/19/09 21:44:20 changed by ip-rob

I should correct myself...this patch modifies both the normal and agressive matching to require an extension or outboundCID in the path.

08/26/09 01:00:10 changed by p_lindheimer

  • status changed from new to closed.
  • confirmation changed from Unreviewed to Need Feedback.
  • resolution set to duplicate.

As a result of #3621 this patch does not apply, but it would appear that #3684 has addressed this. If not, can you please open a new ticket (or get us to re-open this) with a modified patch that takes the changes from #3621 into account? For now I'm going to assume this is a dupe and has been fixed by that change.