-----Original Message-----
From: Nisan Bloch [mailto:[EMAIL PROTECTED]]
 
 At 09:40 AM 10/23/02 +0200, you wrote:
Everytime. that is - everytime the driver returns a message to the bearerbox with a non-terminal error. otherwise the message is discarded anyway.

I can think of one place where this might mess things up, but it is external to Kannel. Kannelers might be have written apps that will resend certain messages on certain error conditions. If the backoff and retry is configurable, it could be disabled in Kannel allowing the apps to handle this. 
I don't think it has anything to do with it, because applications will only receive failed delivery reports when the driver returns the message with a terminal error code, for example - SMSCCONN_MALFORMED. in which case the message will be discarded from the queue and the driver should also return a proper delivery report. the retry effect only goes into action when messages are returned into the queue and would have been retried anyway, when the driver returns an error SMSCCONN_FAILED_TEMPORARLY. 
 
The subject of the email was probably not correct. I should have said "Message max retry and backoff".
 
Essentialy you are correct. with that functionality, driver writers may prefer to fail softly (return with a non terminal error code) on delivery problems instead of retrying themselfs or failing hard (with a terminal error code) and sending a delivery report to the application to handle retries.
 
 The other comment I have without having seen the code, is that it might be usefull to use the dlr mechanism to notify smsbox and hence external apps of this. 
 
Right on the money - messages that are failed internally to Bearerbox (what the max retry cap does) do not generate delivery reports..
 Anychance of posting the code as it is so far?
 
Nothing much to see, its really a tiny patch and not much more then what I already posted, but here it is against current CVS
 
--
Oded Arbel
m-Wise mobile solutions
[EMAIL PROTECTED]
 
+972-9-9581711 (116)
+972-67-340014
 
::..
Life would be so much easier if we could just look at the source code.
 -- Dave Olson
 

Attachment: retry.patch
Description: retry.patch

Reply via email to