jookies / jasmin

Jasmin - Open source SMS gateway
http://jasminsms.com
Other
1.02k stars 549 forks source link

v0.11.1 DLRMapNotFound Issue #1199

Open gartlady opened 4 months ago

gartlady commented 4 months ago

Hi,

I've left a comment on this pull request: https://github.com/jookies/jasmin/pull/1187/files#r1594266529

I recently ran an apt update and got upgraded to v0.11.1. However, it seems that this version introduced a breaking bug related to DLRs. Specifically, _pdu is set to r.request on line 296, causing issues with fetching the response correctly.

I've locally reverted this change back to r.response, which has resolved my DLRMapNotFound issues.

It seems that this code change has not been merged to master yet or released on GitHub, but it has been released via apt (jasmin-sms-gateway/jammy,now 0.11.1 all [installed]).

I can create a PR to revert this change but I wanted to query it here first.

Cheers, Dylan

_Originally posted by @gartlady in https://github.com/jookies/jasmin/pull/1187#discussion_r1594266529_