search

tijder / SmsMatrix

A simple SMS <--> Matrix bridge.
https://matrix.to/#/#smsmatrix:matrix.org
GNU General Public License v3.0
216 stars 36 forks source link

SMS not being sent #35

Closed swedneck closed 5 years ago

swedneck commented 5 years ago

on version 0.0.9 i can recieve messages fine, but sending does not work. This seems to be a regression from a previous version, where it worked fine.

ghost commented 5 years ago

it might be my change https://github.com/tijder/SmsMatrix/pull/31

can you post the error you get?

The change I made above is because messages with more than 160 characters were failing. the app would grab them from Matrix, try to send them and nothing no error, nothing.

The above change uses "sendMultipartTextMessage" which works (for me) with messages less than 160 chars and greater than 160 chars.

swedneck commented 5 years ago

It seems it may have been an issue with synapse, as it crashed shortly after me experiencing this issue. Now that i restarted it i can send sms via matrix just fine.

ghost commented 5 years ago

Cool. If it's not a problem can you close this?

I'm more or less writing my own android app and using what I learn from that to fix this one. Eventually I'll send requests to catch errors and let the user know. I just need to learn how first.