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

mirgation: openvpn should be disabled with all upgrades #630

Open
pmelange opened this issue Nov 26, 2018 · 4 comments
Open

mirgation: openvpn should be disabled with all upgrades #630

pmelange opened this issue Nov 26, 2018 · 4 comments

Comments

@pmelange
Copy link
Contributor

pmelange commented Nov 26, 2018

Irregardless of the version number being upgraded to, the init script for openvpn should be disabled.

Currently only an upgrade to 0.2.0 disables openvpn. Anything after leaves the server enabled.

@SvenRoederer
Copy link
Contributor

What's the problem in disabling or not disabling openvpn?

@SvenRoederer
Copy link
Contributor

disabling OpenVpn on all upgrades will cause loss of bbb-vpn connection and there will be no chance to enable it again from remote.

@pmelange
Copy link
Contributor Author

pmelange commented Dec 2, 2018

The hotplug script takes care of that. The wizard disables the init script too. One of the upgrades disabled it. the init script should generally be disabled and the hotplug script should start openvpn.

The problem is that if the router starts and meshes before it is plugged into the wan port, then openvpn will connect though the smart gateway.

@SvenRoederer SvenRoederer added this to To do in Release 2021.x May 18, 2020
@SvenRoederer
Copy link
Contributor

general issue

To reenable all services is a upstream default of OpenWrt. Working on this (for some other issue / project) I published a patch on the Mailinglist. As this change is not covering all cases it was not accepted yet, but should do for us.

this specific issue

As we don't provide any OpenVPN-ish images anymore by default, this specific issue will not happen anymore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Release 2021.x
  
To do
Development

No branches or pull requests

2 participants