Commit Graph

9 Commits

Author SHA1 Message Date
prlanzarin
24ae746a88 fix(connection-status): jitter causes false positive critical alerts
Jitter evaluation, as an alert trigger, was changed in 3.0 to get the internal
average jitter used in the conn-status component data (which is total jitter
delay divided by jitterbuffer emit events). This was done accidentally and that
metric is _very_ different from the one used in 2.7 (point-in-time jitter from
remote-inbound-rtp/inbound-rtp, highest on the interval, gathered on
/utils/stats.js).  The alert thresholds were preserved, which makes it overly
sensitive in regards to jitter (and thus causes it to be critical whenever the
user is in audio).

Remove jitter as a connection status alert trigger, which fixes the
false positive. The implementation on <= 2.7 is also not ideal - if
anything, it generates false negatives. That's why I'm removing jitter for
the time being since it's ill-suited (at least in the way it's used)
for what we want to achieve.
2024-07-30 18:59:49 +00:00
Tainan Felipe
0e7fe3c688
Change: improve client connection status (#20630) 2024-07-04 17:26:09 -03:00
Ramón Souza
c9ae825c4b update status helper component 2023-12-06 09:12:25 -03:00
Ramón Souza
31add17d9c fix adjust settings button 2023-05-26 15:29:29 -03:00
GuiLeme
81ae4e9ca6 [issue-16954] - Adjusting modal priorities 2023-03-29 11:08:56 -03:00
GuiLeme
9329e194fd [issue-16954] - migration of screenshare unavailable, connectionStatus and randomUser 2023-03-24 17:48:00 -03:00
Joao Victor
4c238392b2 fix(connection status modal): data saving settings button 2022-06-09 12:01:05 -03:00
Ramón Souza
1d1375b3cf move modal component to common folder 2022-02-15 15:54:55 +00:00
Joao Victor
e84cfab5fa refactor: connection status modal UX 2021-11-22 08:25:07 -03:00