Summary:	ASTERISK-03753: [patch] rudimentary support for AOC (Advice of Charge)
Reporter:	Frank Sautter (xylome)	Labels:
Date Opened:	2005-03-23 13:40:37.000-0600	Date Closed:	2008-01-15 15:30:52.000-0600
Priority:	Major	Regression?	No
Status:	Closed/Complete	Components:	Core/General
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) aoc-patch-asterisk-v1.diff ( 1) aoc-patch-asterisk-v2.diff ( 2) aoc-patch-asterisk-v3.diff ( 3) aoc-patch-libpri-v1.diff ( 4) aoc-patch-libpri-v2.diff
Description:	This patch provides basic support for ISDN AOC (advice of charge) messages to asterisk. It decodes and encodes Q.956 AOC ROSE messages (AOC-E_UNITS and AOC CHARGING REQUEST) and and delivers them to chan_zap with the PRI_EVENT_HANGUP and PRI_EVENT_HANGUP_REQ messages. ****** ADDITIONAL INFORMATION ****** The patch is currently MISSING any further embedding into the asterisk software. The idea is to allow forwarding of AOC messages on bridged PRI or BRI channels and logging of the AOC details to the CDR. I'm publishing my 'so far' results to the public because my time in the next few months for asterisk will be a bit limited but i think the patch i save to be published in the CVS the way it is at the moment, so other people could use this code for further developements. this patch is in use since several weeks in a production environment with several thousand calls each day with NO problems.
Comments:	By: Frank Sautter (xylome) 2005-03-23 13:43:46.000-0600 disclaimer is on file By: Russell Bryant (russell) 2005-03-23 14:32:36.000-0600 Reminder sent to mattf I figured I would poke mattf about this one since he has done a lot of work on PRI stuff lately. By: Mark Spencer (markster) 2005-03-23 21:47:27.000-0600 I'm assigning this to matt, but it's unlikely he'll be able to look before next thursday or so. By: Matthew Fredrickson (mattf) 2005-04-01 15:30:39.000-0600 I'm alive and back.  I'm still looking at the complete implementation, but so far I like the changes to the pri_facility.* interface.  I'll give an update this weekend when I have more time. By: Frank Sautter (xylome) 2005-04-04 03:09:04 Reminder sent to mattf hi matt, i just updated the patches to the latest cvs version, so you can merge them in. i hope you like my replacements with the symbolic defined names i made on the hex values in your recent MWI stuff. frank By: Matthew Fredrickson (mattf) 2005-04-04 23:05:41 libpri portion merged.  In your asterisk patch, what the is the "da" country code for? Also, if you're going to open another bug for similar changes, it would probably be better opened under the category of libpri instead of zaptel (I'll probably see it quicker if it is). By: petersv (petersv) 2005-04-05 00:28:16 Denmark? By: Frank Sautter (xylome) 2005-04-05 02:28:08 oh, it seems as if i have overseen this when editing the patchfile. peter svensson is right... this is for danish syntax in sayunixtime. it's a remainder of bug http://bugs.digium.com/bugnote_edit_page.php?bugnote_id=0003239. i uploaded a new patchfile. frank sautter edited on: 04-05-05 02:30 By: Matthew Fredrickson (mattf) 2005-04-05 15:30:54 merged By: Russell Bryant (russell) 2005-04-06 11:32:04 not included in 1.0 By: Digium Subversion (svnbot) 2008-01-15 15:30:52.000-0600 Repository: asterisk Revision: 5412 U   trunk/channels/chan_zap.c ------------------------------------------------------------------------ r5412 | mattf | 2008-01-15 15:30:51 -0600 (Tue, 15 Jan 2008) | 2 lines Merging AOC notifications for bug ASTERISK-3753 ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=5412