Skip to content

Conversation

@danxuliu
Copy link
Member

@danxuliu danxuliu commented Jun 3, 2020

Updated version of #2488

Follow up:

  • Handle out of order answers - This will require some deeper changes and it should not be a very common case, so let's do it in a follow up

How to test

  • Modify the code of the external signaling server to simulate a lot of failures (sorry for the formatting, I did not find a way to use lists and blocks of code)
    • In hub.go
  • Setup the MCU
  • Start a call with a user
  • Join the call with another user or guest

Result with this pull request

The participants are eventually connected despite several offers having failed.

Result without this pull request

The participants are kept trying to connect with each other (unless the first and the second offer succeeded; in that case leave the call and join again until the wrong behaviour is shown).

@danxuliu
Copy link
Member Author

danxuliu commented Jul 2, 2021

/backport to stable22

@danxuliu
Copy link
Member Author

danxuliu commented Jul 2, 2021

/backport to stable21

@danxuliu danxuliu marked this pull request as ready for review July 2, 2021 19:16
Copy link
Member

@marcoambrosini marcoambrosini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

giphy

When a peer is ended it is automatically removed from the list of peers,
so it does not need to be explicitly done.

Signed-off-by: Daniel Calviño Sánchez <[email protected]>
@danxuliu danxuliu force-pushed the send-offer-again-when-own-peer-is-not-initially-connected branch from 2011fc6 to 9c7d1d0 Compare August 5, 2021 10:41
@danxuliu
Copy link
Member Author

danxuliu commented Aug 5, 2021

Rebased on latest master. Tested and works 👍

@danxuliu
Copy link
Member Author

danxuliu commented Aug 5, 2021

/backport to stable22.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants