search

wdoekes / asterisk-chan-dongle

chan_dongle channel driver for Huawei UMTS cards, works with Asterisk 14+
Other
298 stars 104 forks source link

State shows as SMS instead of Free #87

Closed turbozapekanka closed 4 years ago

turbozapekanka commented 4 years ago

Today's build image

Can not receive any SMS message (but can send out just fine) Rolled back to the old version but still the same issue. How to fix this now? Tried to reset modems but didnt help

Update: reverted to old version and rebooted the server and it's back to Free now

wdoekes commented 4 years ago

@turbozapekanka : can you pinpoint which version breaks things for you?

git bisect

magcks commented 4 years ago

I would suggest to test a4a7a345e582ee3b15ca3f6142409e4ec2d5c7ba. There were some changes in the fetching logic in #79 (ba5e7215271283d555fd60083ce0d5d71dafe73e)

turbozapekanka commented 4 years ago

It was on https://github.com/wdoekes/asterisk-chan-dongle/commit/7bdef5e6627eafc7d20a9c893f8bd1b781aec8b7

Will do more tests

ljk4160 commented 4 years ago

I have the same problem,can send out and State displays SMS, When receiving SMS, the log displays SMS is full My device is E180, asterisk version is 13.33 if asterisk v16.9.0 not shown SMS is full,but Can not receive any SMS message I tried to reinstall the OS, but I also encountered the above problems

turbozapekanka commented 4 years ago

I'm using freepbx distro (I beleive it is centos 7 based) with asterisk 13.32.0. Not sure if it makes any difference. the kernel is quite old tho. 3.10 or something

ljk4160 commented 4 years ago

I would suggest to test a4a7a34. There were some changes in the fetching logic in #79 (ba5e721)

use a4a7a34 is ok

magcks commented 4 years ago

Thank you. Could you please also test https://github.com/magcks/asterisk-chan-dongle/commit/868bbd41fbf2911cdfba483e50c8e99a79d84258

ljk4160 commented 4 years ago

Thank you. Could you please also test magcks@868bbd4

WARNING[8392]: at_response.c:1280 at_response_cmgr: [dongle0] Error parsing in coming message: Cannot parse UCS-2

miopa commented 4 years ago

Thank you. Could you please also test magcks@868bbd4

This version seems to work for me (on Asterisk 13.31.0)

tested with E173 11.126.83.00.00 and K3765 11.126.03.09.00

magcks commented 4 years ago

Okay, in this case there probably issues with #79. @hselasky Could you please look over the code and check if there could be some problems with #79?

hselasky commented 4 years ago

I'll have a look later today syncing up with the latest code. The SMS state is not critial. From what I understand it is just a read-only status flag.

hselasky commented 4 years ago

Can you try this one: https://github.com/wdoekes/asterisk-chan-dongle/pull/90

hselasky commented 4 years ago

I'm seeing the same issue with latest version (not mine). Must be a merge issues.

hselasky commented 4 years ago

OK, This revision works for me: 413cad90774f6ef547abf239e6101129bbe4d20d

This revision doesn't: 71e3206122a36f45f097a712a3c8cb611db365c4

Problem is related to missing response to AT+CMGR= which fetch SMS'es.

hselasky commented 4 years ago

Hi, This patch fixes it. It's a regression after the PDU parsing restructure. (No my fault, but now it is fixed :-)

https://github.com/wdoekes/asterisk-chan-dongle/commit/a91275d119fdb5f4e3069692a611ca257c47da33

--HPS

hselasky commented 4 years ago

Please retest #90 !

Thank you!

wdoekes commented 4 years ago

(Or master, as I merged just now.)

magcks commented 4 years ago

Thank you very much @hselasky

turbozapekanka commented 4 years ago

Yes I confirm the state now shows Free but there are some error messages https://github.com/wdoekes/asterisk-chan-dongle/issues/97

turbozapekanka commented 4 years ago

Confirming PR https://github.com/wdoekes/asterisk-chan-dongle/pull/90 fixes this