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

rtsp: rename encryption, protocols, serverKey, serverCert into rtspEncryption, rtspProtocols, rtspServerKey, rtspServerCert #4078

Merged
merged 1 commit into from
Dec 25, 2024

Conversation

aler9
Copy link
Member

@aler9 aler9 commented Dec 25, 2024

No description provided.

…cryption, rtspProtocols, rtspServerKey, rtspServerCert
@aler9 aler9 force-pushed the feature/rtsp-params branch from ac30836 to f49257d Compare December 25, 2024 17:18
Copy link

codecov bot commented Dec 25, 2024

Codecov Report

Attention: Patch coverage is 71.15385% with 15 lines in your changes missing coverage. Please review.

Project coverage is 55.64%. Comparing base (28970b3) to head (f49257d).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
internal/conf/conf.go 60.86% 6 Missing and 3 partials ⚠️
internal/conf/rtsp_transports.go 60.00% 4 Missing ⚠️
internal/conf/rtsp_transport.go 0.00% 1 Missing ⚠️
internal/servers/rtsp/session.go 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4078      +/-   ##
==========================================
- Coverage   55.66%   55.64%   -0.03%     
==========================================
  Files         193      193              
  Lines       19792    19803      +11     
==========================================
+ Hits        11018    11020       +2     
- Misses       7930     7936       +6     
- Partials      844      847       +3     

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

@aler9 aler9 merged commit dfb792e into main Dec 25, 2024
8 checks passed
@aler9 aler9 deleted the feature/rtsp-params branch December 25, 2024 17:26
Copy link
Contributor

github-actions bot commented Jan 3, 2025

This issue is mentioned in release v1.11.0 🚀
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