Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-09-28 Thread Alexander Malysh
...@googlemail.com] On Behalf Of Alexander Malysh Sent: Tuesday, September 14, 2010 1:02 AM To: Michael Zervakis Cc: devel@kannel.org; aguerri...@kannel.org Subject: Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages) Hi Michael, you don't have to alter Msg struct. Just

Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-09-24 Thread Alexander Malysh
To: Michael Zervakis Cc: devel@kannel.org; aguerri...@kannel.org Subject: Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages) Hi Michael, you don't have to alter Msg struct. Just add smscconn_uuid to struct split_parts in gw/msg.h. Thanks, Alexander Malysh

Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-09-15 Thread Michael Zervakis
: Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages) could you please send your patch as attachment ? Thanks, Alexander Malysh Am 14.09.2010 um 16:09 schrieb Michael Zervakis: Hi, I moved smscconn_uuid to struct split_parts. Please check. BR

Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-09-15 Thread Milan P. Stanic
On Wed, 2010-09-15 at 10:52, Alejandro Guerrieri wrote: Send it compressed, Kannel's mail server sends text attachments as part of the body. Maybe your mail client does not interpret it properly because I have got patch as attachment. -- Kind regards, Milan

RE: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-09-15 Thread Rene Kluwen
: devel@kannel.org Subject: Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages) Send it compressed, Kannel's mail server sends text attachments as part of the body. Regards, -- Alejandro Guerrieri aguerri...@kannel.org On 15/09/2010, at 10:06, Michael Zervakis wrote: Hi

RE: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-09-14 Thread Michael Zervakis
Subject: Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages) Hi Michael, you don't have to alter Msg struct. Just add smscconn_uuid to struct split_parts in gw/msg.h. Thanks, Alexander Malysh Am 13.09.2010 um 21:31 schrieb Michael Zervakis: Hi, I

RE: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-09-14 Thread Rene Kluwen
: Alexander Malysh [mailto:malys...@googlemail.com] On Behalf Of Alexander Malysh Sent: Tuesday, September 14, 2010 1:02 AM To: Michael Zervakis Cc: devel@kannel.org; aguerri...@kannel.org Subject: Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages) Hi Michael, you

RE: Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-09-13 Thread Michael Zervakis
: Alexander Malysh [mailto:malys...@googlemail.com] On Behalf Of Alexander Malysh Sent: Sunday, September 12, 2010 1:06 PM To: Michael Zervakis Cc: devel@kannel.org; pon...@appcell.net Subject: Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages) Hi, as I already wrote

Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-09-13 Thread Alexander Malysh
, 2010 1:06 PM To: Michael Zervakis Cc: devel@kannel.org; pon...@appcell.net Subject: Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages) Hi, as I already wrote, this patch is not enough. Please see ML wy... Thanks, Alexander Malysh Am 09.09.2010 um 19

Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-09-13 Thread Alejandro Guerrieri
: Sunday, September 12, 2010 1:06 PM To: Michael Zervakis Cc: devel@kannel.org; pon...@appcell.net Subject: Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages) Hi, as I already wrote, this patch is not enough. Please see ML wy... Thanks, Alexander Malysh

Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-09-13 Thread Alexander Malysh
) Regards, From: devel-boun...@kannel.org [mailto:devel-boun...@kannel.org] On Behalf Of Konstantin Vayner Sent: Thursday, December 17, 2009 12:28 PM To: Alexander Malysh Cc: devel@kannel.org Subject: Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages

Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-09-13 Thread Michael Zervakis
[mailto:aguerri...@kannel.org] Sent: Monday, September 13, 2010 8:25 PM To: Alexander Malysh Cc: Michael Zervakis; malys...@googlemail.com; devel@kannel.org Subject: Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages) The uuid would be different each time you start the service

Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-09-13 Thread Alexander Malysh
Zervakis; malys...@googlemail.com; devel@kannel.org Subject: Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages) The uuid would be different each time you start the service right? What would happen if Kannel is restarted and there were pending parts to send? The new

Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-09-12 Thread Alexander Malysh
...@kannel.org] On Behalf Of Konstantin Vayner Sent: Thursday, December 17, 2009 12:28 PM To: Alexander Malysh Cc: devel@kannel.org Subject: Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages) why remembering smsc-id in sms.smsc_id is not enough? how does smsbox

Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-03-19 Thread Konstantin Vayner
Actually, now that i'm rereading this conversation again... You have to remember SMSC pointer not only SMSC-name/id and then teach all routing parts to respect it... What if kannel gets shut down and then starts up again? Obviousely, smsc pointers are different... I say lets make smsc-id's a

Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-03-19 Thread Alejandro Guerrieri
smsc-id's can't be unique. That would ruin load-balancing for example. smsc-admin-id's could be unique though... (imho, they _should_ be unique, otherwise it's pointless to use it). Regards, Alex -- Alejandro Guerrieri aguerri...@kannel.org On 19/03/2010, at 12:51, Konstantin Vayner wrote:

Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-03-19 Thread Konstantin Vayner
hm.. thats weird i have several smscs in my config named smsc_carriername_mt1 , smsc_carriername_mt2 , ... and i put allowed-smsc-id=smsc_carriername_mt in their config, then submit messages with smsc=smsc_carriername_mt (without numbers) and kannel does load balance between them... or at least

Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-03-19 Thread Alexander Malysh
Am 19.03.2010 um 13:05 schrieb Konstantin Vayner: hm.. thats weird i have several smscs in my config named smsc_carriername_mt1 , smsc_carriername_mt2 , ... and i put allowed-smsc-id=smsc_carriername_mt in their config, then submit messages with smsc=smsc_carriername_mt (without

Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-03-19 Thread Alejandro Guerrieri
You're using separate MO/MT binds, so yes, that's possible. But on some scenarios, you might need the binds to appear as a single bind to your application. e.g., if you're using transceiver binds (or single TX/RX binds for MO/MT) and your application can't handle different smsc names for the

Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-03-19 Thread Alejandro Guerrieri
Yes, that also! :) That's why we added the smsc-admin-id parameter to be able to manage the binds independently from the administration interface: to avoid having to sacrifice the non-uniqueness of smsc-id. Regards, Alex -- Alejandro Guerrieri aguerri...@kannel.org On 19/03/2010, at 13:11,

Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-03-19 Thread Alejandro Guerrieri
I kind of agree into making smsc-admin-id mandatory and unique, though it would probably mean breaking many people's configuration files. Note: If you don't specify an smsc-admin-id, it gets loaded with the smsc-id value, so you'd only need to specify it if you're using duplicated smsc-id's.

Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-03-18 Thread Alexander Malysh
Hi, the possibility for endless loop is equal for each part. So this will not help... We need real fix... Thanks, Alexander Malysh Am 17.03.2010 um 18:04 schrieb Michael Zervakis: Dear Alex, From my experience endless loops from temporary errors usually occur before ever transmitting

RE: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2010-03-17 Thread Michael Zervakis
Dear Alex, From my experience endless loops from temporary errors usually occur before ever transmitting the first message part. It's a very rare occurence to transmit first part and then enter in an endless loop with left parts. A partial fix could be to keep two counters in split_parts

Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2009-12-17 Thread Alexander Malysh
Hi, unfortunately this will not work as expected (the rule is: _all_ parts if multipart message have to be send via the same SMSC)... example: SMSC-A - splits (2 parts) - 1 part sent OK - 2 part get temp. error - you put it into global queue for resend - 2 part sent via SMSC-B -

Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2009-12-17 Thread Konstantin Vayner
so the best option would be to requeue the part via same smsc, right ? cause requeueing all parts may also get extra messages to the handset despite it not being able to reconstruct (not to mention the extra money ;) ) On Thu, Dec 17, 2009 at 11:33 AM, Alexander Malysh amal...@kannel.orgwrote:

Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2009-12-17 Thread Alexander Malysh
Am 17.12.2009 um 10:43 schrieb Konstantin Vayner: so the best option would be to requeue the part via same smsc, right ? yes, but it's not easy todo. You have to remember SMSC pointer not only SMSC-name/id and then teach all routing parts to respect it... cause requeueing all parts may also

Re: [PATCH] fix bug #529 (sms-resend-* ignored for concatenated messages)

2009-12-17 Thread Konstantin Vayner
why remembering smsc-id in sms.smsc_id is not enough? how does smsbox remember routing when i submit a message with predefined smsc id from http (sendsms) ? On Thu, Dec 17, 2009 at 12:10 PM, Alexander Malysh amal...@kannel.orgwrote: Am 17.12.2009 um 10:43 schrieb Konstantin Vayner: so the