Check base64 image validation #1615
Conversation
|
Please check why the UT failed |
Fixed in #1617 |
lmdeploy/vl/utils.py
Outdated
| img = load_image_from_base64(image_url.split(',')[1]) | ||
| try: | ||
| img.load() | ||
| except Exception as e: | ||
| raise ValueError('invalid base64 image') from e |
There was a problem hiding this comment.
load_image_from_base64 这个函数可能会直接抛异常，比如文件头部信息就缺失了。
img = load_image_from_base64('/9j/')
There was a problem hiding this comment.
这里抛异常好呢，还是给个 error msg 然后返回一个空白的图片好呢？
另外这里在下面检查图片完整性比较好，img = Image.open(BytesIO(response.content)), img = Image.open(image_url) 都可能遇到不完整的图片，后面做 img.convert('RGB')的时候就会有问题。
There was a problem hiding this comment.
最理想是服务最开始就做 request 的合法性检验，但是图片的信息好像不太好提取（或者重复提取）。pipeline 感觉可以直接抛异常吧。
@lvhan028 有什么建议吗
There was a problem hiding this comment.
抛异常不太好，会让服务crash。
建议返回错误码和错误信息
There was a problem hiding this comment.
抛异常不太好，会让服务crash。 建议返回错误码和错误信息
异常倒并不会让服务 crash。但是不抛异常，如果让它就继续跑，会让 pipeline 卡住。或者用 @irexyc 的想法，新建一个假图，然后报个 error
There was a problem hiding this comment.
直接返回 None 吗？现在这部分的调用栈比较深了，耦合在 generate 里面，generate 返回的是迭代器。
There was a problem hiding this comment.
最理想是服务最开始就做 request 的合法性检验，但是图片的信息好像不太好提取（或者重复提取）。pipeline 感觉可以直接抛异常吧。 @lvhan028 有什么建议吗
不好在接口处中加判断是么？参数合法性校验，应该发生在收到输入的时候
There was a problem hiding this comment.
不好在接口处中加判断是么？参数合法性校验，应该发生在收到输入的时候
接口处加需要走一遍读图的逻辑。要么就把处理图片 prompt 逻辑放到 generate 外面，不然要读图两次
#1602