fatedier / frp

A fast reverse proxy to help you expose a local server behind a NAT or firewall to the internet.
Apache License 2.0
81.43k stars 12.82k forks source link

增加 procd 脚本与 systemd 服务单元小修改 #1209

Closed deadlineOvO closed 4 years ago

deadlineOvO commented 5 years ago

RT

deadlineOvO commented 5 years ago

emmmm

是为什么没有被合并呢?

fatedier commented 5 years ago

配置中不要使用中文,且尽量只提供必需的内容,不增加不必要的修改,有特殊需求的用户可以根据自己的需要添加新的内容。

deadlineOvO commented 5 years ago

配置中不要使用中文,且尽量只提供必需的内容,不增加不必要的修改,有特殊需求的用户可以根据自己的需要添加新的内容。

好的,我这就改
话说机械翻译应该没问题吧

fatedier commented 5 years ago

我感觉还是增加了很多不必要的部分,不够简单,这一块暂时不做修改吧,后面我再具体跟进一下。

deadlineOvO commented 5 years ago

那个procd与systemd的配置文件还没写完

原因之一是我个人并不会用 git 顺便问一下,root的配置文件是不用加吗?

fatedier commented 5 years ago

因为一套配置肯定是不能满足所有人的需求,所以应该尽量只提供能运行的最基本的配置,而不用考虑到所有情况,可以引导有更多需求的用户,自行去做修改。

deadlineOvO commented 5 years ago

也就是说提交的user部分只留nobody就可以

其他类型的配置可以自己开储存库之类?

fatedier commented 5 years ago

具体如何做,还没有想好,可能提供一个自动化的脚本比较合适,需要再参考一下其他的项目的方案。

deadlineOvO commented 5 years ago

也就是说先交个简易的nobody就成?

deadlineOvO commented 4 years ago

似乎这个pr要被搁置一段时间?

fatedier commented 4 years ago

是的,需要再考虑一下怎么做比较好。

deadlineOvO commented 4 years ago

好的,我知道了

ysc3839 commented 4 years ago

我已经向 openwrt/packages 提交了该项目的 package 并且已被合并 https://github.com/openwrt/packages/pull/9267 还实现了 UCI 配置文件的集成,也已经合并 https://github.com/openwrt/packages/pull/9321

另外需要一个开关来禁用 log 中的 VT 序列 (或者自动检测是否输出到 TTY),现在 log 中看到的是这样的:

Wed Jun 26 17:22:02 2019 daemon.info frps[5082]: 2019/06/26 09:22:02 [1;34m[I] [service.go:139] frps tcp listen on 0.0.0.0:7000[0m
Wed Jun 26 17:22:02 2019 daemon.info frps[5082]: 2019/06/26 09:22:02 [1;34m[I] [root.go:204] Start frps success[0m
deadlineOvO commented 4 years ago

我已经向 openwrt/packages 提交了该项目的 package 并且已被合并 openwrt/packages#9267 还实现了 UCI 配置文件的集成,也已经合并 openwrt/packages#9321

另外需要一个开关来禁用 log 中的 VT 序列 (或者自动检测是否输出到 TTY),现在 log 中看到的是这样的:

Wed Jun 26 17:22:02 2019 daemon.info frps[5082]: 2019/06/26 09:22:02 [1;34m[I] [service.go:139] frps tcp listen on 0.0.0.0:7000[0m
Wed Jun 26 17:22:02 2019 daemon.info frps[5082]: 2019/06/26 09:22:02 [1;34m[I] [root.go:204] Start frps success[0m

话说为什么把启动序列延后了?

ysc3839 commented 4 years ago

@funnypro 我提交时并没有看过你的 pull request,99是我个人认为较合理的值。 你可以手动修改,或者向 openwrt/packages 提 issue。

deadlineOvO commented 4 years ago

@funnypro 我提交时并没有看过你的 pull request,99是我个人认为较合理的值。 你可以手动修改,或者向 openwrt/packages 提 issue。

我设置为50的一个原因是为了让frpc的条目与dropbear启动在相近的时间点 这样更方便进行远程管理

deadlineOvO commented 4 years ago

@funnypro 我提交时并没有看过你的 pull request,99是我个人认为较合理的值。 你可以手动修改,或者向 openwrt/packages 提 issue。

话说提出issue的话,是要在那个项目开一个新的issue吗?

deadlineOvO commented 4 years ago

英语苦手,难受

ysc3839 commented 4 years ago

@funnypro 我觉得没必要纠结这一点点时间,你以这个理由去提 issue 估计也不会被支持,因为不同用户的网络环境不同,有的情况下可能网络还没连接上,设置成 99 是较好的选择。再者,用户自己也可以手动修改。

deadlineOvO commented 4 years ago

@funnypro 我觉得没必要纠结这一点点时间,你以这个理由去提 issue 估计也不会被支持,因为不同用户的网络环境不同,有的情况下可能网络还没连接上,设置成 99 是较好的选择。再者,用户自己也可以手动修改。

话说网络没接上会影响frp的工作吗?

ysc3839 commented 4 years ago

@funnypro 这我就不知道了。

deadlineOvO commented 4 years ago

@ysc3839 不过99感觉还是太晚了…… 话说我提交个pr改可以吗

ysc3839 commented 4 years ago

@funnypro 当然可以,不过能不能被合并我就不知道了。

deadlineOvO commented 4 years ago

@ysc3839 这正是我担心的

fatedier commented 4 years ago

@funnypro https://github.com/gofrp/deploy 新建了一个 deploy 的 repo,期望把识别操作系统,一键部署,Dockerfile 相关的脚本等都放在这个 repo 里,如果你有兴趣可以先参考其他成熟的开源项目的做法,尝试一下。这个 repo 里后续不会再引入相关的内容。

deadlineOvO commented 4 years ago

@funnypro https://github.com/gofrp/deploy 新建了一个 deploy 的 repo,期望把识别操作系统,一键部署,Dockerfile 相关的脚本等都放在这个 repo 里,如果你有兴趣可以先参考其他成熟的开源项目的做法,尝试一下。这个 repo 里后续不会再引入相关的内容。

也就是说那个 systemd 服务单元与 procd 服务全放在这个新的 repo 吗?

fatedier commented 4 years ago

这些配置只是其中的一部分,希望不只是写个配置放在那,而是能根据不同场景加上给用户提供一些简单的交互,就能自动部署,初期可以只考虑常见的系统。

要从一个完整的角度考虑这个问题,而不是简单写一份配置。

deadlineOvO commented 4 years ago

这些配置只是其中的一部分,希望不只是写个配置放在那,而是能根据不同场景加上给用户提供一些简单的交互,就能自动部署,初期可以只考虑常见的系统。

要从一个完整的角度考虑这个问题,而不是简单写一份配置。

一个 wget -O- | bash 之后就能跑的 frp s/c 吗?

fatedier commented 4 years ago

不一定是完全能跑,比如执行完成后,可以提示用户去哪修改配置,该用什么命令来启动,停止。

deadlineOvO commented 4 years ago

不一定是完全能跑,比如执行完成后,可以提示用户去哪修改配置,该用什么命令来启动,停止。

我可以说是要加上简单的安装脚本吗?