search

renmu123 / obsidian-image-auto-upload-plugin

auto upload image with picgo
MIT License
324 stars 34 forks source link

请求添加PicList的跨电脑上传支持 #113

Closed Kuingsmile closed 10 months ago

Kuingsmile commented 10 months ago

根据 https://github.com/Kuingsmile/PicList/issues/103 目前PicList和PicList-Core都已经支持了表单上传feature，使用方式为content-type设置为multipart/form-data，表单字段目前可任意设置，但推荐为files，返回值与原接口相同。

目前直接跨电脑使用时，由于还是通过传list字段的文件路径发送请求，或者不带body体默认上传剪贴板文件，都会失败报错，应该可以判断接口host，不是127.0.0.1的时候，采用表单上传来进行适配，从而支持跨电脑上传。

renmu123 commented 10 months ago

其实是我提的issue，还没做完适配，还要个几天

Kuingsmile commented 10 months ago

一开始就发现是你提的了 反向提回来 哈哈哈 😆

renmu123 commented 10 months ago

image 我觉得config字段不应该返回详细配置，不然安全性太低了，要么做一层加密，实在是不应该明文传输

Kuingsmile commented 10 months ago

image 我觉得config字段不应该返回详细配置，不然安全性太低了，要么做一层加密，实在是不应该明文传输

返回详细配置主要是可以不依赖接口接收端的数据库或者保存的配置信息，做一层加密应该是可以的，我修一下

Kuingsmile commented 10 months ago

@renmu123

修改了下逻辑，现在config字段不明文返回了，而是在服务端做加密后返回，密钥可以由用户设置，另外接口保持了兼容，不需要做额外改动

https://github.com/Kuingsmile/PicList/commit/018971509a2fccffcd4dcae5dbe98cfa61570947 image