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
change to resolve 0.0.0.0:2017 security issue #727
base: main
Are you sure you want to change the base?
Conversation
chore: fix ci
Make inno installer great again!
v2rayA should start after nftables
Some distribution like openwrt should still listen |
Right @cubercsl |
It's the default behavior on openwrt. However, docker users might not that like this change. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
default config value changed to 127.0.0.1 from 0.0.0.0, clean and good
@JamesMackerel . Is it ready to get merged or something else needs to be changed or discussed ? |
I'm OK with it. Thank you! |
This comment was marked as off-topic.
This comment was marked as off-topic.
I don't think it's a good idea to change it, because a lot of people use it with remote management, such as in routers and servers without a desktop environment. There are MAYBE some vulnerabilities. However, we should also carefully consider usage scenafios. |
This modification only changes the default config value, users are free to set the program to listen to 0.0.0.0 or any address if they want. I think the only thing we have to be care of is to mention this change in release note. |
Hi @JamesMackerel @mzz2017 Any update on this PR? |
@SaumyaBhushan Now you can set this in /etc/default/v2raya conveniently~ |
https://github.com/v2rayA/v2rayA/issues/725