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

webrtc: disable UDP when not needed #4176

Merged
merged 1 commit into from
Jan 19, 2025
Merged

webrtc: disable UDP when not needed #4176

merged 1 commit into from
Jan 19, 2025

Conversation

aler9
Copy link
Member

@aler9 aler9 commented Jan 19, 2025

No description provided.

@aler9 aler9 force-pushed the fix/webrtc-udpnet branch from 3fd6834 to c3f0fb7 Compare January 19, 2025 14:36
@aler9 aler9 changed the title Fix/webrtc udpnet webrtc: disable UDP when not needed Jan 19, 2025
Copy link

codecov bot commented Jan 19, 2025

Codecov Report

Attention: Patch coverage is 60.00000% with 6 lines in your changes missing coverage. Please review.

Project coverage is 56.47%. Comparing base (4d98628) to head (c3f0fb7).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
internal/protocols/webrtc/peer_connection.go 60.00% 5 Missing and 1 partial ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4176   +/-   ##
=======================================
  Coverage   56.47%   56.47%           
=======================================
  Files         202      202           
  Lines       20648    20655    +7     
=======================================
+ Hits        11661    11665    +4     
- Misses       8119     8121    +2     
- Partials      868      869    +1     

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

@aler9 aler9 merged commit e86a7a8 into main Jan 19, 2025
8 checks passed
@aler9 aler9 deleted the fix/webrtc-udpnet branch January 19, 2025 14:44
Copy link
Contributor

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

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant