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

2010-09-13 Thread Alexander Malysh
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 forgot to mention that msg->msg_ConnId uuid is not saved in store since > it's useless becaus

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

2010-09-13 Thread Michael Zervakis
Hi, I forgot to mention that msg->msg_ConnId uuid is not saved in store since it's useless because if the process restarts the whole Msg will be resend. At least this solution will apply sms-resend-* directives for long sms. BR, -Original Message- From: Alejandro Guerrieri [mailto

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

2010-09-13 Thread Alexander Malysh
Am 13.09.2010 um 19:25 schrieb Alejandro Guerrieri: > 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 uuid wouldn't match with the pending segments. we have to resend all parts becau

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

2010-09-13 Thread Alejandro Guerrieri
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 uuid wouldn't match with the pending segments. Regards, -- Alejandro Guerrieri aguerri...@kannel.org On 13/09/2010, at 19:16, Alexander Mal

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

2010-09-13 Thread Alexander Malysh
Hi, great idea to use uuid to uniquely identify smscconn. we can use this for http admin as well. Thanks, Alexander Malysh Am 13.09.2010 um 19:04 schrieb Michael Zervakis: > Hi, > > I figured out that we can use the same mechanism as in Msg struct to > uniquelly identify each SMSCconn. In t

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

2010-09-13 Thread Michael Zervakis
Hi, I figured out that we can use the same mechanism as in Msg struct to uniquelly identify each SMSCconn. In the attached patch the smscconn struct was modified to include a uuid_t field and Msg struct to store the smscconn uuid_t if we get a temporary error. Smsc_rout2 was also modified to

Re: Release plan

2010-09-13 Thread Alexander Malysh
I don't think this is a showstopper because this is not a new bug. Thanks, Alexander Malysh Am 13.09.2010 um 16:31 schrieb KangMas Willy: > Yes I agree with it. In my case I need to restart the box early in the > morning daily (I sent around 5 millions MT/day). Need some bug fix on it. > > >

Re: Release plan

2010-09-13 Thread KangMas Willy
Yes I agree with it. In my case I need to restart the box early in the morning daily (I sent around 5 millions MT/day). Need some bug fix on it. sangprabv sangpr...@gmail.com http://www.petitiononline.com/froyo/ On Sep 12, 2010, at 4:18 AM, Alejandro Guerrieri wrote: > I've just tried to rep

Re: Release plan

2010-09-13 Thread Alejandro Guerrieri
I don't think that's the problem here. When the error happened, it didn't matter what kind of content you used. I've tried inserting a lot of identical messages (like "Hello") and about 40-50% of them were lost. Regards, -- Alejandro Guerrieri aguerri...@kannel.org On 13/09/2010, at 16:00, Al

Re: Release plan

2010-09-13 Thread Alexander Malysh
how do you store message body and udh in DB? hex? I ask because you may have some errors due to the missing escaping? And even sqlbox show errors, just make sure you send ACK to bearerbox _after_ successful insert into DB and you will never lose MOs. Thanks, Alexander Malysh Am 13.09.2010 um 15

Re: Release plan

2010-09-13 Thread Alejandro Guerrieri
Yeah, me neither. I've also tried to reproduce it on the weekend using latest Kannel's SVN and couldn't. If you still experience the issue, please let us know the versions you use and I'll re-verify it. Regards, -- Alejandro Guerrieri aguerri...@kannel.org On 13/09/2010, at 15:42, Rene Kluw

RE: Release plan

2010-09-13 Thread Rene Kluwen
By chance, you don't get any like the following message in log-level = 0? MYSQL: Database pool got no connection! DB update failed! Or something similar? == Rene -Original Message- From: devel-boun...@kannel.org [mailto:devel-boun...@kannel.org] On Behalf Of Rene Kluwen Sent: Monday, 13

RE: Release plan

2010-09-13 Thread Rene Kluwen
Do you still encounter that problem? I have been trying to reproduce it, but without any luck. If you have pointers, please share. == Rene -Original Message- From: devel-boun...@kannel.org [mailto:devel-boun...@kannel.org] On Behalf Of KangMas Willy Sent: Monday, 13 September, 2010 15:3

Re: Release plan

2010-09-13 Thread KangMas Willy
Hi, At the user point of view. I think SQLBox is not ready yet, since it still contains bug on MO receiving with meta-data. I'm sure Alex is working on it. sangprabv sangpr...@gmail.com http://www.petitiononline.com/froyo/ On Sep 9, 2010, at 8:12 PM, Alexander Malysh wrote: > Hi together, >

Re: wtls branch merged

2010-09-13 Thread Nikos Balkanas
This apparently is not specific to rc5, but includes all linking to openssl libraries, including kannel's https implementation. Not realy. In https RC5 is not "prefered" encryption algorithm like in WTLS. The RSA 1024 anon key is the most popular option in https. AFAIK it can be used only i

Re: wtls branch merged

2010-09-13 Thread Nikos Balkanas
Thanks for bringing them to my attention. I cannot use -std=gnu99 in Solaris and don't get those. I will compile in Linux and fix them. BR, Nikos - Original Message - From: "Alexander Malysh" To: "Nikos Balkanas" Cc: "Kannel Devel" Sent: Monday, September 13, 2010 11:14 AM Subject:

Re: wtls branch merged

2010-09-13 Thread Milan P. Stanic
On Mon, 2010-09-13 at 07:44, Nikos Balkanas wrote: > Actually nothing changes from the day before. wtls is not enabled by > default. This submission just fixes a broken feature, doesn't > provide a new one. The same way that providers built packages before > (without wtls) the same way they will co

Re: wtls branch merged

2010-09-13 Thread Nikos Balkanas
In total agreement. As already stated, rpms are for the masses, wtls for the few that really need it ;-) BR, Nikos - Original Message - From: "Alexander Malysh" To: "Nikos Balkanas" Cc: "Milan P. Stanic" ; "Kannel Devel" ; "Rene Kluwen" Sent: Monday, September 13, 2010 11:06 AM Su

Re: wtls branch merged

2010-09-13 Thread Alexander Malysh
Hi Nikos, patch commited to svn but we have more warnings wtls related: gcc -std=gnu99 -D_REENTRANT=1 -I. -Igw -g -O2 -DDARWIN=1 -L/Developer/SDKs/MacOSX10.5.sdk/usr/lib -I/Developer/SDKs/MacOSX10.5.sdk/usr/include -D_LARGE_FILES= -I/sw/include/libxml2 -I/sw/include -Wall -Wmissing-prototypes

Re: wtls branch merged

2010-09-13 Thread Alexander Malysh
Hi, I just not even noticed that RC5 was not enabled in openssl by default because fink for OSX does this... Back to this patch, wtls was never enabled by default in kannel and wtls requires RC5 (as far as I know). So the only possible solution would be to ship binary packages without wtls and