Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

希望Allow Insecure默认为true #715

Closed
xieccc opened this issue Apr 22, 2020 · 17 comments
Closed

希望Allow Insecure默认为true #715

xieccc opened this issue Apr 22, 2020 · 17 comments

Comments

@xieccc
Copy link

xieccc commented Apr 22, 2020

很多人配置服务器都是默认配置,这个如果默认为false会导致连接不上

@yfdyh000
Copy link
Contributor

众口难调,怕是需要加一个全局默认值选项。

@xieccc
Copy link
Author

xieccc commented Apr 22, 2020

是的,每家平台这地方都不一样,如果默认为false,,经常有证书的问题
建议是true。。。我也是更新了新版的客户端才发现的

@9h5ej
Copy link

9h5ej commented Apr 22, 2020

希望默认值为false的想法是我提出的,目的是增强安全性。前几天github和部分国外CDN节点被中间人攻击,如果不检查证书有效性容易中招。如果是自建节点,申请一个Let's Encrypt证书很容易,而且是免费的。网上的公益节点也大都用正规证书。
总体看来,证书有效的情况比证书无效的情况多得多,所以建议默认检查证书的有效性以保证安全。

@yfdyh000
Copy link
Contributor

@9h5ej core的默认值是什么,如果不配置是默认允许不安全,那么建议N照做。

@9h5ej
Copy link

9h5ej commented Apr 22, 2020

官网上没说,自己也没测试过。你可以测试一下

@yfdyh000
Copy link
Contributor

@9h5ej 我看core的代码好像是默认true。暂无测试环境。不然N生成配置时默认留空,让core决定。

@xieccc
Copy link
Author

xieccc commented Apr 22, 2020

相当于客户端验签的问题,服务器可以自己设置 是否允许不安全(证书无效)传输
建议 保留客户端默认配置,默认为空
由服务端core决定
并且增加一个全局默认按钮,可以自己调整默认配置

@xieccc
Copy link
Author

xieccc commented Apr 22, 2020

#719

@RPRX
Copy link
Contributor

RPRX commented Apr 23, 2020

@RPRX
Copy link
Contributor

RPRX commented Apr 23, 2020

@xieccc
根据tls的原理,服务端core无法决定客户端的allowInsecure
但关于你开的另一个issue,通过订阅链接来设置allowInsecure可行,需要升级分享链接格式 https://github.com/2dust/v2rayN/wiki/%E5%88%86%E4%BA%AB%E9%93%BE%E6%8E%A5%E6%A0%BC%E5%BC%8F%E8%AF%B4%E6%98%8E(ver-2)

@2dust
Copy link
Owner

2dust commented Apr 23, 2020

最新版已经增加了全局参数,用户自己设置
此值不适合加入到分享属性中,因为是由客户端决定

@RPRX
Copy link
Contributor

RPRX commented Apr 23, 2020

@2dust
建议把“是否允许不安全连接(allowInsecure)”的描述改为“允许不安全连接(allowInsecure)”,去掉“是否”两个字,更准确一些。
全局的改为“TLS默认允许不安全连接(allowInsecure)”,也会更准确一些。

@RPRX
Copy link
Contributor

RPRX commented Apr 23, 2020

@2dust
有些值虽然是客户端决定,但可选范围取决于服务端,比如以下两个场景:

1.服务端使用的是自签证书,若分享/订阅时不带allowInsecure=true,被分享者是用不了的(因为默认值是false),小白可能永远解决不了这个问题

2.外面套了一层tls时已经很安全了,vmess自带的加密方式可以选none,没必要多一层加密损失性能,若分享/订阅时不带security=none,被分享者的security仍是默认值auto,损失性能

所以还是希望考虑一下把这两个值加入分享链接 #672

@2dust
Copy link
Owner

2dust commented Apr 23, 2020

@2dust
有些值虽然是客户端决定,但可选范围取决于服务端,比如以下两个场景:

1.服务端使用的是自签证书,若分享/订阅时不带allowInsecure=true,被分享者是用不了的(因为默认值是false),小白可能永远解决不了这个问题

2.外面套了一层tls时已经很安全了,vmess自带的加密方式可以选none,没必要多一层加密损失性能,若分享/订阅时不带security=none,被分享者的security仍是默认值auto,损失性能

所以还是希望考虑一下把这两个值加入分享链接 #672

文字可改进
分享的属性我们需要仔细规划

@1265578519
Copy link

1265578519 commented Apr 23, 2020

core的默认值我看了是allowInsecure=true,而不是allowInsecure=false。
https://v2ray.com/chapter_02/05_transport.html#tlsobject
你们可以直接在core上生成链接试一下
当然v2rayn最好把分享链接强制加个allowInsecure

@fdngs
Copy link

fdngs commented Apr 23, 2020

@1265578519
请看#724
你发的链接https://v2ray.com/chapter_02/05_transport.html#tlsobject 里没有提到core的默认值是什么
1

https://www.v2fly.org/chapter_02/05_transport.html#tlsobject 里明确说明默认false
2

@RPRX
Copy link
Contributor

RPRX commented Apr 23, 2020

core的默认值我看了是allowInsecure=true,而不是allowInsecure=false。
https://v2ray.com/chapter_02/05_transport.html#tlsobject
你们可以直接在core上生成链接试一下
当然v2rayn最好把分享链接强制加个allowInsecure

我特地下载了v2ray-core的代码,并且追踪了调用过程:

AllowInsecure来自于用户配置,代码内没有显式指定值,所以当留空时是false,与v2fly.org写的默认值相同。v2ray.com很长时间都没有更新过了,况且它也并没有写默认值,不知道你是怎么理解的。

以“_test.go”结尾的几个文件会显式指定值为true,但也只是测试用的

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

No branches or pull requests

8 participants
@yfdyh000 @1265578519 @xieccc @2dust @9h5ej @RPRX @fdngs and others