Re: resend_try and sms_resend_frequency in splitted msgs
"Mi Reflejo" <[email protected]>
| Newsgroups | gmane.comp.mobile.kannel.devel |
|---|---|
| Message-ID | <[email protected]> |
Well, isn't called in a retry. Correctly speaking: Is called in a failure. M On 6/21/06, Mi Reflejo <[email protected]> wrote: > Not really. > This is in handle_split() function that is called by > bb_smscconn_send_failed() so this rutine will be called only on a > retry of a splitted msg. > > Regards, > M > > On 6/21/06, Rene Kluwen <[email protected]> wrote: > > Won't this block sending of other SMS messages as well? > > > > Rene Kluwen > > Chimit > > > > -----Original Message----- > > From: [email protected] [mailto:[email protected]]On > > Behalf Of Mi Reflejo > > Sent: woensdag 21 juni 2006 16:14 > > To: Alexander Malysh > > Cc: [email protected] > > Subject: Re: resend_try and sms_resend_frequency in splitted msgs > > > > > > Yes, agree. > > > > What about check max tries from handle_split() and then resend. > > > > Please check attached patch > > > > M > > On 6/21/06, Alexander Malysh <[email protected]> wrote: > > > Hi Martin, > > > > > > unfortunately this patch will not work as expected and therefore -1 from > > me. > > > > > > With this change you put part of the message into the global queue where > > > it will be retransmitted and also rerouted again. So if you have at > > > least 2 equal SMSCs (egal in kannel config but not the same SMSC) here > > > is the 50/50 possibility that this part will be routed via different > > > SMSC and the whole message will be discarded at handset because of this. > > > > > > The thing with splitted messages is that they must go via the same SMSC. > > > > > > Thanks, > > > Alex > > > > > > P.S. This bug known to me for at least 2 month but didn't find elegant > > > solution yet. The easiest solution would be to just remove resend code > > > if smsc active but it would cause more expenses because the whole > > > message (all parts of the message) will be resend. > > > > > > Mi Reflejo schrieb: > > > > When a msg is splitted and fails, handle_split() is called and every > > > > sms_resend_retry conditions are ignored so msgs are retryed infinite > > > > times. > > > > > > > > I'm attaching a patch to discuss. > > > > > > > > I don't know why handle_split() is calling smscconn_send() and is not > > > > putting msgs in outgoing_sms queue. > > > > > > > > My patch put msg in outgoing_sms and check for resend_try before. > > > > > > > > It's working for me. > > > > > > > > Any comment? > > > > > > > > Martín. > > > > > > > > > > > > > >