SagerNet / sing-box

The universal proxy platform
https://sing-box.sagernet.org/
Other
20k stars 2.38k forks source link

Multi user support for hysteria inbound will break the backward compatibility #216

Closed Gzxhwq closed 2 years ago

Gzxhwq commented 2 years ago

Welcome

Description of the problem

这个变动https://github.com/SagerNet/sing-box/commit/617aba84e437df0a42aa1c5502361b2be8c5bde5 会强行要求现有Hysteria用户去更改服务端的配置文件。会令一部分使用cron job更新服务端的用户不能启动。请问是否可以考虑保留原来的结构,不强制使用多用户,类似SS2022那样?

Version of sing-box

```console $ sing-box version # Paste output here ```

Server and client configuration file

```console # paste json here ```

Server and client log file

```console # paste log here ```
nekohasekai commented 2 years ago

不。

Gzxhwq commented 2 years ago

好的,那至少在更新日志上提一下,我就踩到坑上,查找了一个多小时才发现这里不兼容

nekohasekai commented 2 years ago

会令一部分使用cron job更新服务端的用户不能启

不应该自动更新非稳定版本。

Gzxhwq commented 2 years ago

会令一部分使用cron job更新服务端的用户不能启

不应该自动更新非稳定版本。

这个知道,但是如果正式引入了这个改动,就算稳定版自动更新了也一样会出现卡壳。这是个服务器端的breaking change,引入还是需要慎重考虑。