给gin提交的PR通过了
2025年3月18日
PR1
SaveUploadedFile
函数优化- 修改点1: 文件夹权限,没必要去初始化一个切片
- 修改点2: 对
filepath.Dir
函数的重复使用 - 疑问点:这里遇到初次创建文件夹,因为
umask
的存在,所以os.MkdirAll
的权限是经过掩码后的,所以再加了os.Chmod
。 这会产生一个问题,第一次创建之后os.Chmod
是有必要的,但是之后调用,文件夹已经存在,os.MkdirAll
内有os.Stat
判断不再进行操作,但是os.Chmod
底层至少是没有os.Stat
查看的,大概率会重复没必要的调用底层syscall.chmod
? 这个问题没有解答。 (回归最原始的,我看这个方法的改动增加perm
权限是2024年才加的,很丑陋,真的没必要。)
PR2
- 注释中
X-Real-IP
的大小写问题
PR3 (待合并)
- 使用
Trace
帮我查找到的英文语句问题,哈哈。 也尝试了代码问题,看了几个点,大多是不需要改,有两处可以改,但是也没啥必要。