Attached is a patch to CVS head which makes smsbox sendsms wait until bearerbox has received the SMS, thus avoiding possibility to lose messages if system goes down between http-request and before bearerbox receives the SMS to store it.
The patch also gives better information on what happened to SMS, i.e. accepted for delivery (some smsc driver has accepted it), queued for later delivery (no smsc to take it) and rejected (cannot be handled with current setup) As a side-effect, it makes smsbox just a tiny bit slower. And naturally HTTP sendsms requests 'linger' until bearerbox replies (usually like after 0.00001 seconds...) The patch is basically the same I posted some months ago except that I have removed 'the horrible kludge' :] As there was debate on how this system should be used, this patch now introduces 'immediate-sendsms-reply' boolean config value for smsbox. As it is false by default, it means that this new behaviour is _default_ and is a compatibility breaker if sendsms callers check the exact body of the reply. (I made this default as this is how it should be.) A patch to wait ack from smsc is the next step but a bit more complicated... PS: there is no update to userguide in patch; I will do that if this patch is accepted when I commit it.. -- &Kalle Marjola ::: Development ::: Helsinki ::: Enpocket
