Details
Try to send a video and see video info
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
lib/types/media-types.js | ||
---|---|---|
135 ↗ | (On Diff #47482) | I don't know what format is in the original ffmpeg implementation. It's not documented anywhere and when I've tested it the ffmpeg implementation didn't even return it. Because it's later compared to "mp4" to check if format is correct I'm just returning the file extension. We can run more complicated checks but I think it's good enough for our usecase. |
native/media/video-utils.js | ||
261 ↗ | (On Diff #47482) | The native code can return all those values depending on a platform and a video and they're all h264. |
native/media/video-utils.js | ||
---|---|---|
261 ↗ | (On Diff #47482) | If format wasn’t being set by ffmpeg, wouldn’t this check always fail? What happens if the check fails? If it’s not failing, then perhaps it’s worth understanding what ffmpeg is actually returning. Not sure it matters a lot though |
262 ↗ | (On Diff #47482) | Should this and the above one be factored out to the root level to avoid redeclaring on every invocation? |
native/media/video-utils.js | ||
---|---|---|
261 ↗ | (On Diff #47482) | I checked it and I must have printed a wrong thing because now I can see the format. But still it doesn't tell me much, for mp4 video I've got an array like [mov, mp4, ... a bunch of formats I don't know...]. So I'm leaving it as it is. |