search

buaazp / zimg

A lightweight and high performance image storage and processing system.
http://zimg.buaa.us
BSD 3-Clause "New" or "Revised" License
2.69k stars 402 forks source link

部分PNG格式图片404 #135

Closed kfeng0806 closed 8 years ago

kfeng0806 commented 8 years ago

有一些用户上传了部分PNG图片 在服务器中经过imagemagick转换会出现错误404 经过测试在作者文档中的接口上传后也是显示404 但是将原图下载下载来在PS中重新保存一次PNG再上传即正常 目测可能是部分PNG图片编码或者是图片经过某些软件压缩过后的问题 是否可以考虑一下在图片转换失败的情况下直接返回原始文件而不返回404错误?

buaazp commented 8 years ago

线上服务都会遇到各种各样的损坏图片，以GIF、PNG居多，其实最好的做法是上传时检查，校验图片头部，如果失败则拒绝上传，我们的线上服务、Google Photos都是这么搞的。

zimg目前版本还没有上传是检查策略，已经在写了，下个版本发布。

之所以不能打开失败时返回原图，是因为会导致大小和格式不一致。损坏的图片，就不应该展示。

kfeng0806 commented 8 years ago

404错误确实是因为直接上传了未经验证的损坏图片造成的 在上传服务接口处添加了校验删除问题图片后 已正常运作

buaazp commented 8 years ago

Good