Skip to content

Add CLI flag to parse user timestamps from h264 SEI NALs#767

Open
chenosaurus wants to merge 3 commits intomainfrom
dc/user_timestamp_flag
Open

Add CLI flag to parse user timestamps from h264 SEI NALs#767
chenosaurus wants to merge 3 commits intomainfrom
dc/user_timestamp_flag

Conversation

@chenosaurus
Copy link
Contributor

@chenosaurus chenosaurus commented Feb 12, 2026

adds a --publish-user-timestamp flag that enables the feature in the go server sdk to parse H264 stream for timestamps embedded in SEI NALs.

@chenosaurus chenosaurus changed the title Dc/user timestamp flag Add CLI flag to parse user timestamps from h264 SEI NALs Feb 12, 2026
@chenosaurus chenosaurus requested a review from boks1971 February 12, 2026 06:54
@chenosaurus chenosaurus marked this pull request as ready for review February 12, 2026 06:56
},
&cli.BoolFlag{
Name: "publish-user-timestamp",
Usage: "When publishing H.264/H.265, attach the LKTS user timestamp trailer to each encoded frame (timestamp is sourced from preceding H.264 SEI user_data_unregistered when present)",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mix of H.264/H.265 mention here. Mentions only H.264 SEI in details. Would be good to add H.265 details also.

Also, in join.go flag ☝️ , it only mentions H.264.

Also, join.go commands are deprecated. Would be good to not add new ones there.

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