vidgear icon indicating copy to clipboard operation
vidgear copied to clipboard

Add "rtsps" compatibility

Open jonra1993 opened this issue 1 year ago • 1 comments

Brief Description

This add "rtsps" support. rtsp over TLS

Requirements / Checklist

  • [x] I have read the PR Guidelines.
  • [x] I have read the Documentation.
  • [ ] I have updated the documentation accordingly(if required).

Related Issue

Context

It adds TSL compatibility over RTSP already ffmpeg has this support. With this change it is able to handle this urls rtsps://localhost:8322/client1/camera2.

Types of changes

  • [x] Bug fix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to change)

Screenshots (if available):

image

Before image

After (It works) image

jonra1993 avatar Nov 17 '23 01:11 jonra1993

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (4497fbb) 93.90% compared to head (21d2121) 85.76%. Report is 1 commits behind head on testing.

Additional details and impacted files
@@             Coverage Diff             @@
##           testing     #384      +/-   ##
===========================================
- Coverage    93.90%   85.76%   -8.15%     
===========================================
  Files           16       16              
  Lines         3169     3169              
===========================================
- Hits          2976     2718     -258     
- Misses         193      451     +258     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Nov 17 '23 02:11 codecov[bot]

@jonra1993 Thanks for the PR,. But I've to close it, as you need to raise a Issue first and push changes. Also, push changes to dev branch as I'm currently working on it.

abhiTronix avatar Apr 05 '24 12:04 abhiTronix