fishjam-dev / fishjam

General purpose media server. Supports WebRTC, HLS, RTSP, SIP
https://fishjam-dev.github.io/fishjam-docs/
Apache License 2.0
197 stars 10 forks source link

Move config to struct, fix HlsSkip #130

Closed roznawsk closed 8 months ago

roznawsk commented 8 months ago

Acknowledging the stipulations set forth:

codecov[bot] commented 8 months ago

Codecov Report

Merging #130 (cc32507) into main (6c2f2cb) will decrease coverage by 0.03%. The diff coverage is 93.75%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #130 +/- ## ========================================== - Coverage 86.93% 86.90% -0.03% ========================================== Files 58 59 +1 Lines 1056 1054 -2 ========================================== - Hits 918 916 -2 Misses 138 138 ``` | [Files](https://app.codecov.io/gh/jellyfish-dev/jellyfish/pull/130?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jellyfish-dev) | Coverage Δ | | |---|---|---| | [lib/jellyfish/room.ex](https://app.codecov.io/gh/jellyfish-dev/jellyfish/pull/130?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jellyfish-dev#diff-bGliL2plbGx5ZmlzaC9yb29tLmV4) | `86.79% <100.00%> (-0.09%)` | :arrow_down: | | [lib/jellyfish\_web/api\_spec/hls.ex](https://app.codecov.io/gh/jellyfish-dev/jellyfish/pull/130?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jellyfish-dev#diff-bGliL2plbGx5ZmlzaF93ZWIvYXBpX3NwZWMvaGxzLmV4) | `100.00% <ø> (ø)` | | | [lib/jellyfish\_web/api\_spec/room.ex](https://app.codecov.io/gh/jellyfish-dev/jellyfish/pull/130?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jellyfish-dev#diff-bGliL2plbGx5ZmlzaF93ZWIvYXBpX3NwZWMvcm9vbS5leA==) | `100.00% <ø> (ø)` | | | [lib/jellyfish\_web/controllers/room\_controller.ex](https://app.codecov.io/gh/jellyfish-dev/jellyfish/pull/130?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jellyfish-dev#diff-bGliL2plbGx5ZmlzaF93ZWIvY29udHJvbGxlcnMvcm9vbV9jb250cm9sbGVyLmV4) | `100.00% <100.00%> (ø)` | | | [lib/jellyfish\_web/controllers/room\_json.ex](https://app.codecov.io/gh/jellyfish-dev/jellyfish/pull/130?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jellyfish-dev#diff-bGliL2plbGx5ZmlzaF93ZWIvY29udHJvbGxlcnMvcm9vbV9qc29uLmV4) | `100.00% <100.00%> (ø)` | | | [lib/jellyfish/room/config.ex](https://app.codecov.io/gh/jellyfish-dev/jellyfish/pull/130?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jellyfish-dev#diff-bGliL2plbGx5ZmlzaC9yb29tL2NvbmZpZy5leA==) | `94.11% <94.11%> (ø)` | | | [lib/jellyfish/room\_service.ex](https://app.codecov.io/gh/jellyfish-dev/jellyfish/pull/130?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jellyfish-dev#diff-bGliL2plbGx5ZmlzaC9yb29tX3NlcnZpY2UuZXg=) | `84.61% <90.00%> (-1.81%)` | :arrow_down: | ------ [Continue to review full report in Codecov by Sentry](https://app.codecov.io/gh/jellyfish-dev/jellyfish/pull/130?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jellyfish-dev). > **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=jellyfish-dev) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://app.codecov.io/gh/jellyfish-dev/jellyfish/pull/130?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jellyfish-dev). Last update [6c2f2cb...cc32507](https://app.codecov.io/gh/jellyfish-dev/jellyfish/pull/130?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jellyfish-dev). 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=jellyfish-dev).