Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

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

Closed
Kuingsmile opened this issue Oct 11, 2023 · 5 comments
Closed

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

Kuingsmile opened this issue Oct 11, 2023 · 5 comments

Comments

@Kuingsmile
Copy link

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

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

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

@Kuingsmile
Copy link
Author

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

@renmu123
Copy link
Owner

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

@Kuingsmile
Copy link
Author

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

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

@Kuingsmile
Copy link
Author

@renmu123

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

Kuingsmile/PicList@0189715
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@renmu123 @Kuingsmile