Add ffmpeg package to n8n Docker image for media processing#118
Add ffmpeg package to n8n Docker image for media processing#118SashkoMarchuk wants to merge 1 commit intomainfrom
Conversation
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
🤖 CodeRabbit AI Review AvailableTo request a code review from CodeRabbit AI, add CodeRabbit will analyze your code and provide feedback on:
Note: Reviews are only performed when |
📝 WalkthroughWalkthroughThe Dockerfile.n8n is updated to include ffmpeg as an additional package in the apk package installation command, expanding the set of system dependencies bundled in the Docker image alongside git. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 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 |
|
There was a problem hiding this comment.
🧹 Nitpick comments (1)
Dockerfile.n8n (1)
13-13: Pinffmpegto a specific version for reproducible builds.
gitis pinned to2.49.1-r0, butffmpegis unpinned. This inconsistency means the image isn't fully reproducible — a rebuild could pull a different ffmpeg version with breaking changes or vulnerabilities.Proposed fix
First, find the current version available in the base image's Alpine release:
#!/bin/bash # Check which Alpine version the base image uses and the available ffmpeg package version docker run --rm n8nio/n8n:1.109.2 sh -c "cat /etc/alpine-release && apk info ffmpeg 2>/dev/null || apk list ffmpeg 2>/dev/null || apk search -e ffmpeg"Then pin accordingly, e.g.:
- apk add --no-cache git=2.49.1-r0 ffmpeg && \ + apk add --no-cache git=2.49.1-r0 ffmpeg=<version> && \



Click Up Task: https://app.clickup.com/t/86c7xzdpc
Summary by CodeRabbit