ascorbic / unpic

Universal image CDN translator
https://unpic.pics/lib
280 stars 31 forks source link

Working with Cloudinary video thumbnails #112

Open seancdavis opened 7 months ago

seancdavis commented 7 months ago

There are two different ways to convert a Cloudinary video URL to a thumbnail image of the video:

In both cases, the assetType captured by this regex is video, which leads to an unsupported error.

Video thumbnails feel like a valid use case, and I'm happy to do the work to add the support.

But what's the right logic? Is it too opinionated to say if it's a video, then we check for a valid image extension on the URL?