Hi Nikos,
ok got it...
We have such discussions many times already and we always got to
decision that kannel can't and should't
support things that are not standard except it easy to integrate and
has no impact on the code readability,
security etc.
I don't think we have to patch DLR parsing in SMPP due to this clear
SMSC bug. This is some homegrown SMSC
just don't accept quasi standard. This SMSC should be fixed and not
kannel...
Thanks,
Alexander Malysh
PS: I think we should have this configurable and regex group matching
should do it...
Am 22.10.2009 um 19:24 schrieb Nikos Balkanas:
Sure.
http://www.mail-archive.com/[email protected]/msg17659.html
----- Original Message -----
From: Alexander Malysh
To: Nikos Balkanas
Cc: [email protected] Devel
Sent: Thursday, October 22, 2009 5:58 PM
Subject: Re: Patch: smsc_smpp.c
can you please give me link to this discussion?
Thanks,
Alexander Malysh
Am 22.10.2009 um 13:17 schrieb Nikos Balkanas:
Hi,
Oh boy. You missed all the fun :-)
Original email by Latitude Test on 13/10/2009, where DELIVERD DLRs
are misreported by kannel as failures (type 2). There was quite a
discussion about whether it was a kannel bug, or an out of spec
DLR. In the end it was a consensus that kannel needed a patch.
Bottom of the line: Spec is very loose at this point about DLR
fields. Kannel expects either an exact format (sscanf) or it
reverts to a more flexible old style search. Problem is that in the
search it assumes that the value of each field is followed by
space, and that is not necessarily true (if field is last in DLR).
Seikath also said that he has a couple of cases like that.
BTW, I have asked Latitude to test it, because I cannot, but he
seems to get disappeared after creating all this stir :-(
BR
Nikos
----- Original Message -----
From: Alexander Malysh
To: Nikos Balkanas
Cc: [email protected] Devel
Sent: Thursday, October 22, 2009 11:08 AM
Subject: Re: Patch: smsc_smpp.c
Hi Nikos,
could you please explain why we need this patch?
Thanks,
Alexander Malysh
Am 19.10.2009 um 18:30 schrieb Nikos Balkanas:
Hi,
A trivial patch, that should be able to handle all DLRs as long as
they keep formal tags.
Please test.
BR,
Nikos<smsc_smpp.diff>
