tuituidan / team-nav

前后端完整的网址导航服务，相较一般的靠配置生成的网址导航服务，本系统具有更强的后台管理能力，技术上主打一个轻量级，无任何依赖服务。

MIT License

356 stars 65 forks source link

直接通过url上传图标，导致的图片相关接口问题 #37

Open oneisall8955 opened 8 months ago

oneisall8955 commented 8 months ago

版本

v1.1.4

重现流程

图标管理中，打开上传选择框中，非选择本地文件而是粘贴icon url如：https://lf3-cdn-tos.bytescm.com/obj/static/xitu_juejin_web/static/favicons/favicon-32x32.png

如图

上传后，接口提示成功，但是图片名称会携带xxx[1].png，但是icon无法预览
此时，删除接口，修改图标名称接口均发生异常删除接口异常截图：

修改名称异常截图

在服务器中，检查到存在该文件从服务器中下载本地文件是正常的

修复建议

应该是api使用restful api风格，在spring生态中，path路径存在[ ]这种非英文或数字字符导致的异常

可能日志级别原因，后端日志并没有异常

oneisall8955 commented 8 months ago

2.0.0版本错误提示如下：

tuituidan commented 8 months ago

@oneisall8955 报错我后面看看，不过你的这个在选择文件框中填url的上传方式是我从来没想到过的，文件上传一般三种操作： 1、点击上传按钮选择本地磁盘文件 2、在本地磁盘选择图片Ctrl+C，然后在本系统设置页面上Ctrl+V 3、在网页上找到一张在线图片，右键选择复制图片，然后在本系统设置页面Ctrl+V，就能粘贴上传这个线上图片了

oneisall8955 commented 8 months ago

@tuituidan 谢谢，希望可以优化删除这种文件名的文件，虽然不影响使用，但是显示在列表里面是空白的图标。目前是用方式1，不知道还有2和3方式来上传。

我这种奇怪的上传方式是偶然发现的，并且有些web系统确实支持了，应该是利用了选择文件框填写url格式自动下载的特性

wangbf19740612 commented 8 months ago

@oneisall8955 @tuituidan url的方式可能会造成一些安全问题，个人建议尽量不要用哈