Hi I tested the patch and for now I don’t see any problem with it. Before adding it to trunk, can you update the rating docs and record all changes in 1 patch using darcs? After that I can apply it directly on trunk with your credentials.
These instructions to submit the patch also apply on CDRTool: http://sipsimpleclient.org/projects/sipsimpleclient/wiki/SipSupport You don’t need to open a ticket however, just tell me after you send the patch. -- Tijmen de Mes AG-Projects On 26 januari 2014 at 02:08:50, Duane Larson ([email protected]) wrote: Ugh.... Nevermind. My "Max Duration" was set to 6 on the Destinations rate setup. I set it to zero and I think things are looking better now. Think I'm done for the day. Good work David! Really appreciate the patch. On Sat, Jan 25, 2014 at 6:56 PM, Duane Larson <[email protected]> wrote: Not sure my "price" is being calculated correctly. My audio.outbound should be $0.005 My audio.inbound should be $0.0035 Here is an example of a call that lasted 2:06 minutes and is audio.outbound Increment=6 MinDuration=6 MaxDuration=6 ConnectFee=0.0000 CallId=dc1e32b1ba18ff91a3 67aa8df81e8e3c@0:0:0:0:0:0:0:0 Span=1 Duration=6 DestId=1 default Profile=USA_Default Period=weekend Rate=USA_Default Interval=0-24 Cos t=0.0050/60 Price=0.0005 PriceIn=0.0000 Here is an example of a call that lasted 1:33 minutes and is audio.inbound Increment=6 MinDuration=6 MaxDuration=6 ConnectFee=0.0000 CallId=2e0334513a34964e7c0 [email protected]:5060 Span=1 Duration=6 DestId=1 default Profile=USA_Default Period=weekend Rate=USA_Default Interval=0-24 C ost=0.0035/60 Price=0.0003 PriceIn=0.0000 Looks like no matter what the duration is for the inbound calls it costs $0.0003 and the outbound calls all cost $0.0005. On Sat, Jan 25, 2014 at 6:24 PM, Duane Larson <[email protected]> wrote: David/Tijmen/Adrian, It is working for me too. Both inbound and outbound are being recognized and the different rates are being applied. I will keep looking at it while more calls are being made to make sure there are no unforeseen issues. On Fri, Jan 24, 2014 at 5:35 AM, Tijmen de Mes <[email protected]> wrote: Hi David, Thanks for the patch. If have some time Monday I will analyze it and give you some feedback so we include this in CDRTool. Besides the rating, I’ve to check if the code that now sets the the ‘route’ for the CDRs and if there are no problems. -- Tijmen de Mes AG-Projects On 23 januari 2014 at 21:21:26, David M. Lee ([email protected]) wrote: I’ve got a first attempt at a patch: https://gist.github.com/leedm777/8585690 To get this to work, your CDR’s will have to record the application subtype. For me, I set $avp(s:call_class) = “audio.inbound” or “audio.outbound” as appropriate in the routes, and added Sip-Application-Type=$avp(s:call_class) to “radius_extra”. Billing rates will be matched on the full application, allowing different rates depending on the application subtype. It works with the simple testing I’ve been able to do on my desktop. Feedback, of course, is greatly appreciated. Duane - Does the patch work for you? Adrian - Any objections/concerns with this approach? If it looks good, I’ll work on updating the docs. Thanks! -- David M. Lee Digium, Inc. | Software Developer 445 Jan Davis Drive NW - Huntsville, AL 35806 - USA Check us out at: www.digium.com & www.asterisk.org On Jan 23, 2014, at 4:43 AM, Adrian Georgescu <[email protected]> wrote: > I think it would be a good idea. > > Adrian > > On 22 Jan 2014, at 16:58, David Lee (digium) <[email protected]> wrote: > >> Duane Larson wrote: >>> I have been playing with CDRTool for a while but I am not sure if it is >>> possible to rate Origination (Inbound) calls differently than Termination >>> (Outbound) calls from my SIP Provider. For Origination I pay 0.0035 and >>> for Termination I pay 0.005. Keep in mind these costs are for destination >>> "1”. >> >> I've recently run into nearly the same situation. >> >> I tried overloading the Sip-Application-Type field, but it's limited >> by the supportedApplicationTypes array in cdr_generic.php. There also >> seems to be hard coded logic for the different application types. >> >> I think I will patch CDRTool so that you can have subtypes of >> application types (audio.inbound, audio.outbound, etc.). This would >> allow the billing rates to be a bit more specific for these >> situations. >> >> Thoughts? >> -- >> David M. Lee >> Digium, Inc. | Software Developer >> 445 Jan Davis Drive NW - Huntsville, AL 35806 - USA >> Check us out at: www.digium.com & www.asterisk.org >> _______________________________________________ >> Users mailing list >> [email protected] >> http://lists.opensips.org/cgi-bin/mailman/listinfo/users > > _______________________________________________ > Users mailing list > [email protected] > http://lists.opensips.org/cgi-bin/mailman/listinfo/users _______________________________________________ Users mailing list [email protected] http://lists.opensips.org/cgi-bin/mailman/listinfo/users _______________________________________________ Users mailing list [email protected] http://lists.opensips.org/cgi-bin/mailman/listinfo/users -- -- *--*--*--*--*--* Duane *--*--*--*--*--* -- -- -- *--*--*--*--*--* Duane *--*--*--*--*--* -- -- -- *--*--*--*--*--* Duane *--*--*--*--*--* -- _______________________________________________ Users mailing list [email protected] http://lists.opensips.org/cgi-bin/mailman/listinfo/users
_______________________________________________ Users mailing list [email protected] http://lists.opensips.org/cgi-bin/mailman/listinfo/users
