NextChat icon indicating copy to clipboard operation
NextChat copied to clipboard

fix the problem of error at download the screenshot picture when there are some pics in messages history

Open sunreview opened this issue 1 year ago • 7 comments

之所以报错是因为 消息记录中的图片不属于私域中的图片 ,需要在toPNG之前 用一个后台接口转换为私域的图片地址,然后toPNG则不会报错可以正常下载 #3553

下面是 修复后的功能截图

动物 快乐

sunreview avatar Dec 17 '23 15:12 sunreview

lgtm, sadly I can't cherry pick commit this branch because bug still not fixed yet

H0llyW00dzZ avatar Dec 17 '23 16:12 H0llyW00dzZ

lgtm

i think it is solved ... you can add gpt-4v model and test it ...it is solved ... these pic tells everythong 下载

sunreview avatar Dec 17 '23 23:12 sunreview

it we don't add the transferimg api and there's the pics in the chat history , toPNG method will go some error because the pic don't belong to this domain .

that's why we need to add the transferimg api

sunreview avatar Dec 25 '23 08:12 sunreview

it we don't add the transferimg api and there's the pics in the chat history , toPNG method will go some error because the pic don't belong to this domain .

that's why we need to add the transferimg api

yeah because of cors issue, been known it

H0llyW00dzZ avatar Dec 25 '23 09:12 H0llyW00dzZ

I cherry pick pr this to my fork https://hackerchat.btz.sh/#/changelog

H0llyW00dzZ avatar Dec 25 '23 09:12 H0llyW00dzZ

@tomridder is attempting to deploy a commit to the NextChat Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Feb 15 '24 02:02 vercel[bot]

Your build has completed!

Preview deployment

github-actions[bot] avatar Feb 15 '24 02:02 github-actions[bot]

thanks a lot for your works, it's a good way to solve cross-domain problems, but it would lead to a ssrf

Dean-YZG avatar Mar 27 '24 10:03 Dean-YZG