Skip to content

Deny access localhost and localnet#70

Closed
sorz wants to merge 2 commits into
shadowsocks:masterfrom
sorz:master
Closed

Deny access localhost and localnet#70
sorz wants to merge 2 commits into
shadowsocks:masterfrom
sorz:master

Conversation

@sorz
Copy link
Copy Markdown

@sorz sorz commented Oct 10, 2013

Shadowsocks make user get through not only firewall on client side, but also server's firewall.
Since Shadowsocks server is often setted up with other services, such as MySQL. It may increase the risk.

(Though the implement may not be good...)

@clowwindy
Copy link
Copy Markdown

See the comments in shadowsocks/shadowsocks-nodejs#70

@sorz
Copy link
Copy Markdown
Author

sorz commented Oct 14, 2013

我的意思是,SS 用户可以通过代理连接 127.0.0.1,即让 SS Server 连接 127.0.0.1。由于服务器防火墙一般直接信任 local loop [来源请求],这相当于使服务器的全部 TCP 端口对 SS 用户暴露。这点比较容易被忽视 [来源请求],从而形成安全隐患。

@clowwindy
Copy link
Copy Markdown

有很多人把 shadowsocks Python 版放在路由器或者一台特定的电脑上跑,比如我。

改防火墙配置比改 shadowsocks 配置更适合解决这个问题。

@sorz
Copy link
Copy Markdown
Author

sorz commented Oct 14, 2013

呃.. 你可能还没理解我的意思?
我想说的是 server.py 这边的问题。比如一台 VPS 同时运行 server.py 和 MySQL,MySQL bind 在 127.0.0.1 不对外开放。
SS 用户便可以通过 server.py 访问到原本不对外开放的 MySQL 服务。但通常这不是在 VPS 上架设 SS 的目的。
我认为很难通过修改防火墙配置解决这个问题。

@clowwindy
Copy link
Copy Markdown

Sorry,和其它几个 issue 弄混了。这个我再想想。感觉这样配置太复杂了。

@lilydjwg
Copy link
Copy Markdown

On Mon, Oct 14, 2013 at 07:21:08AM -0700, BlueN wrote:

呃.. 你可能还没理解我的意思?
我想说的是 server.py 这边的问题。比如一台 VPS 同时运行 server.py 和 MySQL,MySQL bind 在 127.0.0.1 不对外开放。
SS 用户便可以通过 server.py 访问到 MySQL 服务。但通常这不是在 VPS 上假设 SS 的目的。
我认为很难通过修改防火墙配置解决这个问题。

插一句,如果有 SELinux 的话还是很简单的~

另外,iptables 我记得是有按用户 ID 处理的模块的。

Best regards,
lilydjwg

Linux Vim Python 我的博客:

http://lilydjwg.is-programmer.com/

A: Because it obfuscates the reading.
Q: Why is top posting so bad?

@sorz
Copy link
Copy Markdown
Author

sorz commented Oct 14, 2013

我也不是很确定是否必要。我是最近想私下公开一台服务器才遇到的这个问题。如果 SS 设计之初就是个人使用绝对信任用户的,这个问题就不存在了,最多在 README 加上一条风险提示…

@clowwindy
Copy link
Copy Markdown

MySQL 一般单机使用的时候用 unix socket,方便权限管理。

@sorz sorz closed this Oct 15, 2013
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