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

Call or message not received when application is removed from recent apps #772

Open
ognjenns opened this issue Dec 12, 2017 · 7 comments
Open
Assignees
Milestone

Comments

@ognjenns
Copy link
Contributor

When Olympus application is not active, its in the recent apps, and push notification is sent; The RCDevice is initialized and call/message is received. When application is not in the recent apps, the RCDevice is initialized but call/message is not received.

@atsakiridis
Copy link
Collaborator

Managed to reproduce the issue. Scenario:

  1. Open Android Olympus and setup push notifications; verify it's good by testing an incoming call after hitting home button
  2. Kill Android Olympus from recents
  3. Try a second incoming call (from web olympus) and at this point there's no INVITE coming in and in the caller side we get:
2017-13-12 17:11:34.359 INFO SIP message received: SIP/2.0 404 Cannot complete P2P call
To: <sip:[email protected]>;tag=85454476_04905836_57a5b08a_7d9f64ce
Via: SIP/2.0/WSS 141.237.152.226:55125;branch=z9hG4bK-313035-9a58204ba8ac03d52f536655f026d734;rport=55125;received=141.237.152.226
CSeq: 2 INVITE
Call-ID: 1513177888911
From: "totos" <sip:[email protected]>;tag=1513177889567
Server: Restcomm 8.3.0-54
Contact: <sip:52.55.231.191:5060;transport=tcp>
Content-Length: 0

Analysis:

  • I see that REGISTER is successful and even after ~8 seconds I receive the OPTIONS from RC without issues
  • This means that the signaling/transport channel seems clear
  • The only weird thing is Connection attempt already in progressbut still this might show up in good scenarios as well
  • Also the fact that OPTIONS comes properly after so many seconds means that signaling channel in Olympus/SDK is still open/healthy
  • It doesn't seem to be android/sdk issue. Server somehow decides not to forward the call and returns 404 to the caller
  • I'd suggest that you try to debug with @agafox to see why the 404 is issued from the server

Here are the logs

@atsakiridis atsakiridis assigned ognjenns and unassigned atsakiridis Dec 13, 2017
@ognjenns
Copy link
Contributor Author

Also to add that settings for the ICE Server where somehow mess up.
screen shot 2017-12-13 at 4 49 37 pm

@atsakiridis
Copy link
Collaborator

I'm pushing this issue to next sprint because it needs to be investigated in a different way. Here are the conclusions we reached today with @agafox and @ognjenns:

  • The problem here is the 3 second timeout in the server side, which means that if it takes more than 3 seconds for the REGISTER to come in after the push has gone out, then the call is cancelled.
  • This problem can also occur on a regular push scenario where the App isn't killed; it's just more easily reproducible if the App has been killed as it takes more time.

Action plan:

  • As a first step we need to investigate if we can make the client come up faster. Mainly profile it and see if we have any big issues that we can fix easily
  • If that isn't possible we need to see how we can improve the server design so that this delay isn't needed.

@atsakiridis
Copy link
Collaborator

Assigning to myself now that the scope of this issue changed to profiling the SDK

@atsakiridis atsakiridis assigned atsakiridis and unassigned ognjenns Dec 22, 2017
@atsakiridis
Copy link
Collaborator

Did some profiling and provided my input here

@atsakiridis
Copy link
Collaborator

There's on action on our side on this. But I'm leaving this open so that when server side is ready we can verify that the issue is fixed

@SlobodanPrijic
Copy link

@atsakiridis any update regarding this issue? Seems that calls are still cancelled to early (current cloud service).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants