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

Receive less logs than registered in phones log #232

Closed
leonardoporpora opened this issue Jan 19, 2018 · 20 comments
Closed

Receive less logs than registered in phones log #232

leonardoporpora opened this issue Jan 19, 2018 · 20 comments

Comments

@leonardoporpora
Copy link

I receive much less messages through signal than the logs captured on the phone. I receive 2 photos but on the phone there were 16 photos

@leonardoporpora
Copy link
Author

I’m using the latest beta

@lukeswitz
Copy link
Collaborator

lukeswitz commented Jan 19, 2018

Try changing setting "Notification Time Interval" to 0. Restart and see if that does it. It's built in to not annoy you with all the notifications.

@leonardoporpora
Copy link
Author

It’s not this the issue I have tried but nothing changed

@n8fr8
Copy link
Member

n8fr8 commented Jan 19, 2018

So, even if you set the internal to 0, you still don't receive all the notifications?

@n8fr8 n8fr8 added the bug label Jan 19, 2018
@n8fr8 n8fr8 added this to the Current Sprint milestone Jan 19, 2018
@leonardoporpora
Copy link
Author

Exactly

@leonardoporpora
Copy link
Author

@n8fr8 I have tried again and I got 20 logs in the phone and only 3 photos I have received

@n8fr8
Copy link
Member

n8fr8 commented Jan 22, 2018

It could be that Signal might be throttling the amount of content coming through. I will look into it.

@leonardoporpora
Copy link
Author

hey @n8fr8 have u found anything?

@leonardoporpora
Copy link
Author

still have the same issue

@leonardoporpora
Copy link
Author

i have received just 3 logs but on the phone there were 28..... @lukeswitz

@lukeswitz
Copy link
Collaborator

Let's see how it goes with an update to the signal-cli. Will report back.

@leonardoporpora
Copy link
Author

leonardoporpora commented Mar 25, 2018

thanks :)

@leonardoporpora
Copy link
Author

@lukeswitz
Copy link
Collaborator

lukeswitz commented Apr 3, 2018

Good find, but there isn't any limiting going on using the guardianproject/signal-cli-android that I can see. It handles a pretty intense flow of messages when it's active:

A 'message bomb' will work, but not on the first run. You can try testing using:

  • fresh launch,
  • recording a dummy session with at least one event
  • next session should hit each alert.

Thanks for your help. Will update if things work better on the new branch.

@lukeswitz
Copy link
Collaborator

I found none haven-side, but the Signal app does use limiting. Signal logs indicate rate limit exceeded upon the first attempt to register a number if Haven has been used previously/in parallel.

It's possible the acceptance rate of messages in Signal follows this standard, which makes sense. It will lock users out from registering devices without user feedback (as I've found in multi-device support tests).

@leonardoporpora
Copy link
Author

But instead of kill the upload of the photo it could make a queue, is it possible so that i would receive all photos?

@lukeswitz
Copy link
Collaborator

Possible. Check sent:database event count and then dispatch when available. Applicable to losing net connectivity as well.

@leonardoporpora
Copy link
Author

I have still the same problem and this is a bad issue cause I saw that there were 65 events on my phone but I received only 7 photos in which none where shown, seeing the logs I had a person that I could recognize... But if he takes my phone and I haven't received the photos on my primary phone ...

@lukeswitz
Copy link
Collaborator

lukeswitz commented Aug 27, 2018

Same issues persisting here across all devices tested. The problem isn’t with Signal, Haven is asking a bit too much of the integration.

Signal docs mention that messages aren’t completely independent when sent in a small window, which could easily explain why 60 are coming through as only six.

Perhaps event logging needs an asynchronous method to handle the db entry & alerts (as bitmap conversion footprint is currently higher than any prior release)

Are you able to view video events remote/local? I noted adding even 10ms latency improved continuity across logs and alerts for images.

Lots of opportunities across the board to rewrite the persistence layer and solve issues on legacy & flagship devices. Thanks again for continued feedback!

@n8fr8
Copy link
Member

n8fr8 commented Feb 11, 2019

I've opened a new ticket to address new feature work to implement a complete remote access/sync mechanism

#370

Signal is just meant for notification, not full remote backup

@n8fr8 n8fr8 closed this as completed Feb 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants