jitsi / jicofo

JItsi COnference FOcus is a server side focus component used in Jitsi Meet conferences.
Apache License 2.0
314 stars 350 forks source link

fix: Handle timeout from jibri as an error. #1144

Closed bgrozev closed 4 months ago

codecov[bot] commented 4 months ago

Codecov Report

Attention: Patch coverage is 0% with 11 lines in your changes are missing coverage. Please review.

Project coverage is 28.98%. Comparing base (fb29dc8) to head (1e5181a).

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/jitsi/jicofo/pull/1144/graphs/tree.svg?width=650&height=150&src=pr&token=wdwr26KPKY&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi)](https://app.codecov.io/gh/jitsi/jicofo/pull/1144?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi) ```diff @@ Coverage Diff @@ ## master #1144 +/- ## ============================================ - Coverage 28.99% 28.98% -0.02% Complexity 478 478 ============================================ Files 130 130 Lines 7912 7915 +3 Branches 1087 1087 ============================================ Hits 2294 2294 - Misses 5345 5348 +3 Partials 273 273 ``` | [Files](https://app.codecov.io/gh/jitsi/jicofo/pull/1144?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi) | Coverage Δ | | |---|---|---| | [...main/java/org/jitsi/jicofo/jibri/JibriSession.java](https://app.codecov.io/gh/jitsi/jicofo/pull/1144?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi#diff-amljb2ZvL3NyYy9tYWluL2phdmEvb3JnL2ppdHNpL2ppY29mby9qaWJyaS9KaWJyaVNlc3Npb24uamF2YQ==) | `40.00% <0.00%> (-0.50%)` | :arrow_down: | ------ [Continue to review full report in Codecov by Sentry](https://app.codecov.io/gh/jitsi/jicofo/pull/1144?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://app.codecov.io/gh/jitsi/jicofo/pull/1144?dropdown=coverage&src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi). Last update [fb29dc8...1e5181a](https://app.codecov.io/gh/jitsi/jicofo/pull/1144?dropdown=coverage&src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi).