[jitsi-dev] [jitsi/jitsi-videobridge] Endpoint connection monitoring (#272)


#1

This PR adds endpoint connection status monitoring which sends notifications over data channels when connectivity status changes. Endpoint connection status is considered inactive when there is no media or ICE related traffic on any of it's channels. This is tracked in new 'lastTransportActivityTime' timestamp added to every Channel. It is update whenever new RTP packets is received or when ICE "consent freshness" check succeeds(it is now send every 3 seconds).
You can view, comment on, or merge this pull request online at:

  https://github.com/jitsi/jitsi-videobridge/pull/272

-- Commit Summary --

  * Configure values for "consent freshness" checks
  * Make 'broadcastMessageOnDataChannels' public
  * Add 'lastTransportActivityTime' to Channel

-- File Changes --

    M pom.xml (4)
    M src/main/java/org/jitsi/videobridge/Channel.java (52)
    M src/main/java/org/jitsi/videobridge/Conference.java (2)
    M src/main/java/org/jitsi/videobridge/IceUdpTransportManager.java (2)
    M src/main/java/org/jitsi/videobridge/RtpChannel.java (7)
    M src/main/java/org/jitsi/videobridge/osgi/JvbBundleConfig.java (14)

-- Patch Links --

https://github.com/jitsi/jitsi-videobridge/pull/272.patch
https://github.com/jitsi/jitsi-videobridge/pull/272.diff

···

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/272


#2

just curious, what's the goal of this work? is it to be able to detect clients who have timed out more quickly? that's definitely something we've been wanting to do...glad to see the consent check interval shortened, that was one of the main blockers.

···

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/272#issuecomment-234610633


#3

So right now we have this blurred screen effect added to the UI of Jitsi Meet when ICE disconnected state is triggered. But that happens only locally for the user and others do not see that. After this PR the JVB will send notification to everyone in the conference when connectivity issues are detected. There is ongoing work on handling those events in the lib-jitsi-meet and the meet app.

···

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/272#issuecomment-234611636


#4

@paweldomas pushed 3 commits.

93de555 Implement endpoint connection status monitoring
52ddab7 Add "SCTP connection ready" event
dcfee2e Send info about disconnected endpoints for new peers

···


You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/272/files/b3bdcd81193af19555e78c925475923393fdde16..dcfee2eb310baaadd72476c9a3526d3624b2e66f


#5

Merged #272.

···

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/272#event-792326495