Use first proper GIF frame for thumbnail

leafo / magick

Lua bindings to ImageMagick for LuaJIT using FFI

401 stars 79 forks source link

Use first proper GIF frame for thumbnail #24

Closed maxgorovenko closed 8 years ago

maxgorovenko commented 8 years ago

Patch for #22

maxgorovenko commented 8 years ago

Second commit fixes #5

leafo commented 8 years ago

Sorry for the late response. I think it's better to have coalesce be a manual step. It's since been added. Regarding gifs:

the coalesce method now exists, but I don't recommend it for user uploaded images. It's going to use up a lot of memory, and potentially be an attack vector. I created this library to hande extracting the first frame from a gif: https://github.com/leafo/giflib