Skip to content

Conversation

@awsong
Copy link
Contributor

@awsong awsong commented Apr 20, 2023

通过打开非认证tcpip localAPI通道,增加tailscale cli远程连接daemon的能力,对调试封闭环境的客户端(比如Network Extension)有帮助。

awsong added 2 commits April 20, 2023 15:19
enabled.

Add remote localAPI service to debugMux(Windows version), so that a remote tailscale
client can connect to it for control.
@gps949
Copy link
Member

gps949 commented Apr 20, 2023

我的两点意见:
1、非认证的TCP监听会带来额外的安全风险;因此至少不能放到发布版本中,如果确实有必要,需要按照条件编译方式而非运行时变量或参数处理;
2、目前暂时没看到这方面明确的需求,不建议合并入主分支,可以以独立分支存在;

@awsong
Copy link
Contributor Author

awsong commented Apr 20, 2023

第一个我加的时候同时加了个debugMode的开关,默认是关的。

@gps949
Copy link
Member

gps949 commented Apr 20, 2023

第一个我加的时候同时加了个debugMode的开关,默认是关的。

我comment中的第1条后半句

如果确实有必要,需要按照条件编译方式而非运行时变量或参数处理;

指的就是这一点

@awsong
Copy link
Contributor Author

awsong commented Apr 20, 2023

go 的条件编译只能整个文件,加这个server如果放单独文件,处理就复杂多了。
TS默认的 debugMux 就是运行时参数,这个server就是加到了debugMux里面的。

@gps949
Copy link
Member

gps949 commented Apr 20, 2023

TS原本的debugMux功用和你添加的目的不大一样,而且是命令行运行参数控制的。
如果觉得条件编译比较复杂,也可以改成传参的形式

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants