Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SMS are randomly delivered on concurrent sending #51

Open
robo-w opened this issue Apr 14, 2018 · 1 comment
Open

SMS are randomly delivered on concurrent sending #51

robo-w opened this issue Apr 14, 2018 · 1 comment
Labels

Comments

@robo-w
Copy link

robo-w commented Apr 14, 2018

Message with same content should be sent to multiple recipients:

  • Javascript client iterates over recipients, calls send URL for each recipient

If the calls are executed at the same time, only the first address gets a message, all other messages are lost and are not visible in the outgoing message list.

If the calls are queued and delayed by 500 to 2000 milliseconds between each call, most of the recipients get messages, but some get duplicate messages. In the list of the outgoing messages, all recipients are visible.

@m4r10k
Copy link
Member

m4r10k commented Apr 14, 2018

Yes, you are correct. It works better if you use a delay during your send calls or you can use multiple devices to spread the text messages for example. The code is far from being perfect at this point. Are you using multiple WIS instances or are you using a singular WIS? The WIS is holding the database and is responsible for the operations against it.

Nowadays it would be better to use something like a distributed ETCD with appropriate concurrency locking instead of self managed multiple Sqlite db's. In the past we thought about it to use something like https://github.com/rqlite/rqlite alternatively.

So yes, there would be a lot of work to resolve this issue sustainably.

@m4r10k m4r10k added the bug label Apr 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants