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-0600Date Closed:2008-01-15 15:30:52.000-0600
Versions:Frequency of
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.


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.


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


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


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