Hi again, any objections on this from others?
On 2/22/06, Dziugas Baltrunas <[EMAIL PROTECTED]> wrote: > Hi, > > thanks for the fix. However, some nuances about the whole billing > procedure remain. > > 1) Currently (after the fix) a call to external script will be made > for each of the recipients (billing-library functionality). Having > some prepaid customer trying to send a message, this might lead to > incorrect check for the balance, if, say, customer has a sufficient > balance only to send MMS for 3 recipients, but originally there were, > say 4 recipients. This will work as expected if only one call to the > external script providing all recipients or at least count of them as > script's parameters would be made. > > 2) I productional environments IMHO real billing should be made only > after the message was sent (or queued for delivery). I guess for this > purpose prov-server-notify-script was dedicated. However, currently > parameters provided to this script are not sufficient to implement > billing as the script is called per message, but not per recipient. > Some additional parameters such as message id and size of the message > would be also useful here. > > I would like to get some more opinions on the billing logic and how it > was initially designed to work. I'm mainly referring to simple but > efficient NowSMS billing (acutally callbacks) model, which is > described here (see MMSSend PreAuth Callback and MMSSend Accounting > Callback): > > http://www.nowsms.com/support/bulletins/TB-NOWSMS-001.htm > > Currently callbacks for Sending an MM in mbuni are as follows: > > 1. Call to billing-library to check if there is sufficient balance to > send an MM for all the recipients. > 2. For each of the MM recipients there is a call to resolver-library > to check whether is local or foreign recipient or MM shall be sent to > VASP. > 3. prov-server-notify-script is called in case of MM was queued for > delivery. Here real billing should happen. > > Correct me if I'm wrong. > > On 2/21/06, Paul Bagyenda <[EMAIL PROTECTED]> wrote: > > Fixed I hope and updated to CVS. > > > > On Feb 21, 2006, at 16:35, Dziugas Baltrunas wrote: > > > > > Hi, list, > > > > > > what was the purpose for the code in mmsc/ > > > mms_billing_shell.c::mms_billmsg(): > > > > > > for (i=0;i<list_len(to);i++) { > > > Octstr *s; > > > s = octstr_format("%s '%s' '%s'", octstr_get_cstr(script), > > > octstr_get_cstr(from), octstr_get_cstr(list_get(to, i))); > > > > > > if (s) { > > > system(octstr_get_cstr(s)); > > > octstr_destroy(s); > > > } > > > } > > > > > > ? > > > > > > As you can see return value from the script currently is ignored so we > > > should reconsider this and either pass all the recipients in one pass > > > or check if WEXITSTATUS(system(octstr_get_cstr(s))) is always > > > possitive for each of the recipients. > > > > > > -- > > > Dziugas > > > > > > _______________________________________________ > > > Devel mailing list > > > Devel@mbuni.org > > > http://mbuni.org/mailman/listinfo/devel_mbuni.org > > > > ----------------------------------------------- > > Trek the Rwenzori's. Or just see them online - http:// > > www.rwenzori.com/gallery.htm > > > > > > _______________________________________________ > > Devel mailing list > > Devel@mbuni.org > > http://mbuni.org/mailman/listinfo/devel_mbuni.org > > > > > -- > Dziugas > -- Dziugas _______________________________________________ Devel mailing list Devel@mbuni.org http://mbuni.org/mailman/listinfo/devel_mbuni.org