Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

srt: process connection requests in parallel (#3382) #3534

Merged
merged 2 commits into from
Jul 5, 2024
Merged

srt: process connection requests in parallel (#3382) #3534

merged 2 commits into from
Jul 5, 2024

Conversation

aler9
Copy link
Member

@aler9 aler9 commented Jul 5, 2024

Fixes #3382

Copy link

codecov bot commented Jul 5, 2024

Codecov Report

Attention: Patch coverage is 45.94595% with 20 lines in your changes missing coverage. Please review.

Project coverage is 56.69%. Comparing base (afec0cd) to head (58d66b1).
Report is 3 commits behind head on main.

Files Patch % Lines
internal/servers/srt/conn.go 36.66% 19 Missing ⚠️
internal/servers/srt/server.go 80.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3534      +/-   ##
==========================================
- Coverage   56.75%   56.69%   -0.07%     
==========================================
  Files         190      190              
  Lines       19164    19122      -42     
==========================================
- Hits        10877    10841      -36     
+ Misses       7460     7457       -3     
+ Partials      827      824       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@aler9 aler9 merged commit 342c257 into main Jul 5, 2024
6 checks passed
@aler9 aler9 deleted the fix/srt branch July 5, 2024 20:17
Copy link
Contributor

github-actions bot commented Jul 7, 2024

This issue is mentioned in release v1.8.4 🚀
Check out the entire changelog by clicking here

@github-actions github-actions bot locked and limited conversation to collaborators Jan 10, 2025
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extra connections when attempting to read an SRT stream with incorrect credentials
1 participant