Skip to content

localAddr add#70

Closed
denghongcai wants to merge 1 commit into
shadowsocks:masterfrom
denghongcai:master
Closed

localAddr add#70
denghongcai wants to merge 1 commit into
shadowsocks:masterfrom
denghongcai:master

Conversation

@denghongcai
Copy link
Copy Markdown

No description provided.

@clowwindy
Copy link
Copy Markdown
Contributor

Since very few people need this, I won't merge this feature to main stream. More options cause confusion.

@clowwindy clowwindy closed this Oct 9, 2013
@denghongcai
Copy link
Copy Markdown
Author

对于在学校使用静态IP的人来说是有必要的,否则会被外界扫描到,localAddr可以作为一个可选项,这样就不会造成困惑了

@clowwindy
Copy link
Copy Markdown
Contributor

可以先把这个功能作为一个 fork,直到有更好的方法不让配置文件变得更臃肿又能实现这个功能。

@clowwindy
Copy link
Copy Markdown
Contributor

最简单的方法是增加一条防火墙规则。

目前凡是可以用防火墙 / iptables 做到的事情我都不打算加到 shadowsocks 里。

@tshwangq
Copy link
Copy Markdown

thanks bro.

@denghongcai
Copy link
Copy Markdown
Author

作为一个非必填可选项也不行么,我会保持这个fork

@clowwindy
Copy link
Copy Markdown
Contributor

@denghongcai 我想尽量保持功能和配置项的精简。

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