fix(docker): add ffmpeg to viewer image for video thumbnails#172
Conversation
The viewer container was missing ffmpeg, causing all video thumbnail requests to return 404. Image thumbnails worked because Pillow was present, but video extraction requires ffmpeg.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughPR adds ffmpeg system tooling to the viewer Dockerfile for video thumbnail extraction, and bumps package version from 7.11.0 to 7.11.1 across project metadata and package exports. ChangesVideo Thumbnail Support & Patch Release
🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #172 +/- ##
=======================================
Coverage 92.55% 92.55%
=======================================
Files 23 23
Lines 6599 6599
=======================================
Hits 6108 6108
Misses 491 491
🚀 New features to boost your workflow:
|
|
🐳 Dev images published!
The dev/test instance will pick up these changes automatically (Portainer GitOps). To test locally: docker pull drumsergio/telegram-archive:dev
docker pull drumsergio/telegram-archive-viewer:dev |
Summary
ffmpegtoDockerfile.viewerso video thumbnail generation works in the viewer container_check_ffmpeg()returned FalseTest plan
Summary by CodeRabbit