[keyldev]

• nobuffer discards packets read during stream probing, including the keyframe some NVRs send at session start; later frames reference it, so nothing decoded (VideoToolbox errors, grey in software)
• Applies to the live decoder on every platform and the web live remuxer

Summary

Related

Type

• Bug fix
• Feature
• Refactor / cleanup
• Docs / CI
• Other:

Checklist

• Builds with 0 warnings (TreatWarningsAsErrors=true).
• Tests pass (dotnet test); new Core logic has unit tests.
• No layering violation — App references Core only (Infrastructure / Video / Devices wired via DI in a head).
• Scope stays within one phase (didn't pull work from a later phase's "Не входит").
• README / docs updated if public commands, options, or setup changed.

Platforms tested

• Windows
• Linux
• macOS
• Android
• iOS
• CI build only

Screenshots / notes

[qodo-free-for-open-source-projects]

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Tip of the day

💡 Did you know, you can import rules and skills only from the repositories your team follows

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

[qodo-free-for-open-source-projects]

PR Summary by Qodo

Preserve opening RTSP keyframes in live video

🐞 Bug fix 🕐 Less than 10 minutes

AI Description

• Remove nobuffer from native and web RTSP opens so stream probing retains opening keyframes.
• Prevent dependent frames from failing to decode on affected NVR streams.

Diagram

graph TD
  Camera["RTSP camera"] --> Session["Native video session"] --> Decoder["Frame decoder"]
  Camera --> Ffmpeg["Web FFmpeg process"] --> Mp4["Fragmented MP4"] --> Hub["Live stream hub"]

Loading

High-Level Assessment

Removing nobuffer at both RTSP open points directly addresses the lost opening keyframe. The other latency and timeout settings remain in place; no broader pipeline change is warranted.

Files changed (2) +7 / -3

Bug fix (2) +7 / -3

FfmpegVideoSession.csRetain probe packets in the native RTSP session +4/-1

Retain probe packets in the native RTSP session

• Removes the 'fflags=nobuffer' input option so packets read during stream probing remain available to the decoder. Adds a comment explaining the opening-keyframe failure seen with some NVRs.

src/OpenIPC.Viewer.Video/Pipeline/FfmpegVideoSession.cs

LiveFfmpeg.csRetain opening keyframes in the web live stream +3/-2

Retain opening keyframes in the web live stream

• Removes '-fflags nobuffer' from the FFmpeg process arguments used for web RTSP streams. Keeps the transport and socket-timeout arguments unchanged.

src/OpenIPC.Viewer.Web/Api/LiveFfmpeg.cs