-
Notifications
You must be signed in to change notification settings - Fork 194
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
Keep-alive connections #169
Comments
I don't know how ChatSecure works, but it keeps an always active connection to the server (by the way OTR needs just that), which does indeed drain your battery, I doubt it's zero impact. |
I am aware of all of that.
That is exactly the point which should be tested! Maybe keeping an active connection is less energy wasting than reconnecting that often, since reconnecting takes much more data to be transferred than simple ping mechanism.
I do already have configured those timeouts so that Kontalk is connected the maximal amount of time and those things I described in the first post still happen, which shouldn't be the case. |
Always-on connection was the policy we adopted in the first place, and many complained about extreme battery draining. It's very hard to not drain energy while a connection is alive. I will add an option "None" to the idle connection timeout (so Kontalk will never disconnect if the user decides so). |
First of all I don't have Google Services on my phone.
Since the new version is not keeping the app awake in the background, which is how it really should be, there are times when messages do not arrive/arrive delayed when Kontalk is in the background. When such a case happens all messages can be retrieved by manually opening the app. How to reproduce (seems not to work all the time):
The battery life now is terrific and this shouldn't change. I assume Kontalk is not connecting anymore or something like that. Maybe ChatSecure is a good reference to solve this. Messages in Chatsecure always get delivered immediately and the app has zero battery impact.
The text was updated successfully, but these errors were encountered: