gopeed icon indicating copy to clipboard operation
gopeed copied to clipboard

feat: add permission related settings

Open 1lkei opened this issue 1 year ago • 6 comments

给docker镜像添加了设置用户用户组umask功能

1lkei avatar Feb 16 '24 13:02 1lkei

感谢感谢,我还堵在高速上呢,明天看看

monkeyWie avatar Feb 17 '24 13:02 monkeyWie

感谢感谢,我还堵在高速上呢,明天看看

没事,我也是第一次提交pr,明天你看看有没有问题

1lkei avatar Feb 17 '24 13:02 1lkei

我试了有问题,给gopeed设置普通用户权限后他会自己在根目录创建Downloads目录并设置为下载目录,手动改回/root/Downloads也没权限,即使我设置/root/Downloads所有者跟程序一样也不行,其他应该没问题了

1lkei avatar Feb 18 '24 06:02 1lkei

我考虑了下,为了不同用户统一安装目录,下个版本把docker里的默认下载目录改成/opt/gopeed/download,然后这个pr再改一改就可以了

monkeyWie avatar Feb 18 '24 13:02 monkeyWie

根目录/Downloads作为默认下载目录可行吗?反正都要映射出来

michaelauto avatar Feb 18 '24 16:02 michaelauto

根目录/Downloads作为默认下载目录可行吗?反正都要映射出来

这不是根本解决办法,等新版修改默认下载地址后再说吧

1lkei avatar Feb 19 '24 08:02 1lkei

抱歉久等了,Docker构建默认下载目录改成/app/Download了,麻烦再更新下 pr 哦

monkeyWie avatar Mar 24 '24 13:03 monkeyWie

抱歉久等了,Docker构建默认下载目录改成/app/Download了,麻烦再更新下 pr 哦

等我一下,时间有点久我有点忘了,先研究研究

1lkei avatar Mar 24 '24 14:03 1lkei

抱歉久等了,Docker构建默认下载目录改成/app/Download了,麻烦再更新下 pr 哦

看看这样应该ok了

1lkei avatar Mar 24 '24 15:03 1lkei

LGTM

monkeyWie avatar Mar 25 '24 00:03 monkeyWie