| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
| |
|
|
|
| |
It is much cleaner if we check first for disabled OpenVPN instances, before diving into all the other checks. Note, the final openvpn_resync() does call other routines that finally check if the instance is disabled, but there are so many checks here for various conditions to be met before calling openvpn_resync that it looks better (safer) to bail out early if the instance is disabled.
|
|
|
|
| |
Commit https://github.com/pfsense/pfsense/commit/f33dcc5c79c54af7daf91a81cfdd7f489e8cb67c reversed the logic sequence when testing if $resync_needed - the individual tests were changed from "==" to "!=" and so on, but the conjunction also need to be changed - "or" needs to be "and". I had noticed that VPNs on some gateway groups of mine didn't failover recently, but hadn't gone looking for the problem until now.
This might help bug #3243 - it will probably now make the OpenVPN resync on every interface/gateway change for the cases of CARP VIPs and VLANs mentioned in that bug report. At least that will be better than not resyncing at all.
|
|
|
|
|
|
|
| |
- try_lock: used for trying to get an EXCLUSIVE lock for a specified timeout by default of 5
- unlock_force: which just releases any locks held on a specified lock
Use this new functions on rc.openvpn to avoid spurious stale locks around.
|
|
|
|
| |
locks around
|
|
|
| |
The argument passed to rc.openvpn can be a comma-separated list of gateways - not just 1 gateway. Enhance the code to loop and process each gateway.
|
| |
|
|
|
|
| |
twice and don't checking openvpn client
|
|
|
|
|
| |
Various fixups to make this work. Now I can:
- Unplug an interface, any OpenVPN servers/clients in a gateway group using that interface are restarted and come up on the highest tier available interface. OpenVPN servers/clients that are only on that interface go down, of course.
- Plug in the cable again, any OpenVPN servers/clients in a gateway group using that interface are restarted and come up on the now-highest tier available interface (i.e. they fail back if the interface that just came up is higher tier). OpenVPN servers/clients that are only on that interface now come up.
|
| |
|
| |
|
|
|
|
| |
related items
|
|
|
|
|
| |
If there are OpenVPN servers but not clients, this warning is emitted:
Warning: Invalid argument supplied for foreach() in /etc/rc.openvpn on line 55
This fixes handles that case, and the case of OpenVPN clients but no servers.
|
| |
|
|
|
|
| |
interface is actually a gateway group name. Otherwise we skip.
|
|
|
|
| |
we don't need to do that here.
|
| |
|
|
|