Skip to content

Support Encoding and Decoding MPEG-TS #1007

Merged
aler9 merged 7 commits intobluenviron:mainfrom
EricHenry:decode-encode-mpegts
Feb 28, 2026
Merged

Support Encoding and Decoding MPEG-TS #1007
aler9 merged 7 commits intobluenviron:mainfrom
EricHenry:decode-encode-mpegts

Conversation

@EricHenry
Copy link
Contributor

As part of bluenviron/mediamtx#5476 add the ability to encode and decode MPEG-TS packets.

@codecov
Copy link

codecov bot commented Feb 26, 2026

Codecov Report

❌ Patch coverage is 70.37037% with 24 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.09%. Comparing base (e0a98c8) to head (7f2928d).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
pkg/format/rtpmpegts/encoder.go 62.26% 16 Missing and 4 partials ⚠️
pkg/format/mpegts.go 66.66% 2 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1007      +/-   ##
==========================================
- Coverage   83.17%   83.09%   -0.09%     
==========================================
  Files         124      126       +2     
  Lines       11676    11757      +81     
==========================================
+ Hits         9712     9769      +57     
- Misses       1449     1467      +18     
- Partials      515      521       +6     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@aler9
Copy link
Member

aler9 commented Feb 28, 2026

i did the following changed:

  • decoder and encoder were refactored to work with detached TS packets ([][]byte) instead of concatenated TS packets ([]byte), and in order to remove the limitation that MaxPayloadSize was forced to be a multiple of mpegtsPacketSize
  • fuzzing
  • linting

then i merged. Thanks.

@aler9 aler9 merged commit 9cad698 into bluenviron:main Feb 28, 2026
6 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants