Endpoints for managing recordings

[eravellaSC]

This is a WIP pull request (I am still working on the whole feature) that I am posting to ask if it makes sense to have this API extension. The aim here would be having some interface to manage the recorded streams via the REST API, allowing operations as

• listing
• deletion
• streaming (still need to be added, I am trying to figure out how to stream from file or if it is easier to just create a stream and run an FFMPEG script)

Does this feature make sense?

[bog]

This is something I'd be very interested in.

[eravellaSC]

I added tests and refactored some code. I think that the steaming back functionality is being addressed in #2906 (and it's still achievable through the already existing API and the use of hooks). I will wait for review and if there are things that need to be adjusted I will work on them.

[codecov]

Codecov Report

Attention: 60 lines in your changes are missing coverage. Please review.

Comparison is base (cf68fe4) 53.18% compared to head (6090fc6) 51.86%.
Report is 29 commits behind head on main.

Files	Patch %	Lines
internal/api/api.go	0.00%	60 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2901      +/-   ##
==========================================
- Coverage   53.18%   51.86%   -1.32%     
==========================================
  Files         124      130       +6     
  Lines       11448    12682    +1234     
==========================================
+ Hits         6089     6578     +489     
- Misses       4905     5611     +706     
- Partials      454      493      +39     

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

[t3therdev]

Hi @eravellaSC playback is released in 1.5. I think your suggested API enhancements would work well also.

[aler9]

sorry for not including this in 1.5 but i had no time for reviewing it. Will do ASAP.

[eravellaSC]

Not a problem for me. Let me know if further work is needed / if I can help with something

[aler9]

Unfortunately this PR lacks some critical features, including:

• handling segments of paths that use a recordPath value different from the one in defaultPaths
• retrieving path names
• segment management independent from file paths on disk
• pagination

i've created a new patch here: #2988

[github-actions]

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

[eravellaSC]

@aler9 I'm sorry for making you do additional work, I am not very used to golang and I am still studying the internals of MediaMTX. Thanks for the new release though.

[aler9]

thank you for providing the idea behind the feature.