derive codec from file extension for remotion lambda

[uragirii]

Fixes #1248. npx remotion lambda derives the codec from the file extension provided by the user

---

IssueHunt Summary

Referenced issues

This pull request has been submitted to:

• #1248: npx remotion lambda render does not derive the codec from the file extension

---

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated
remotion	✅ Ready (Inspect)	Visit Preview	Oct 7, 2022 at 0:06AM (UTC)

[JonnyBurger]

Thanks a lot @uragirii! 🙌

This is an improvement that I am willing to merge, to be eligible for the $100 bounty though, the acceptance criteria I posted in #1248 should also pass.

[uragirii]

@JonnyBurger how can I make this test pass?

[JonnyBurger]

@uragirii The output for this test is pretty bad, I'll try to at least fix the logging tomorrow so it will give a proper error message!

[JonnyBurger]

@uragirii I have pushed some better error handling, now it's much clearer why the test fails!

[uragirii]

@JonnyBurger seems like tests are wrong! haha!
In transparent-webm.test.ts the outName is "out.mp4" but should end in webm. Similar thing for gif.test.ts the outName is "out.mp4" but should end in gif.

If you want i can fix the tests. Also, the pr should be ready to be reviewed ;)

[JonnyBurger]

🥸

[JonnyBurger]

Fixed, let's see!

[JonnyBurger]

Did some refactors myself since there were already quite a few bugs before and there are many factors for deciding codec and image format!

Nonetheless, excellent work! For statistics, how long do you estimate did it take for you to work on this PR?

[uragirii]

Thanks for merging, and fixes my errors. The amount of time ig is 1:30-2hrs