search

jookies / jasmin

Jasmin - Open source SMS gateway
http://jasminsms.com
Other
991 stars 540 forks source link

DLR for SMS with more than one segment #1157

Closed Herneto closed 3 months ago

Herneto commented 8 months ago

Fixes

1136

Description

I was experiencing the issue #1136 where I didn't receive trought HTTP the SMSC's DLR for sms with more than one segment.

It was happening because it always expect the SMSC's DLR for the first pdu what works for sms with one segment.

I just fixed so that SMSC's DLR is expected for the first pdu for sms with one segment and expected for the last pdu for sms with more than one segment.

farirat commented 7 months ago

Hello @Herneto Am running CI tests and let's hope this would pass them.

farirat commented 7 months ago

Hey @Herneto please take a look to the failed tests.

BlackOrder commented 6 months ago

Hi, @Herneto Just rollback https://github.com/Herneto/jasmin/blob/6eb0f0cbcb4e4f3126b54005dc963b6247da6015/jasmin/managers/listeners.py#L393 I think it will pass then