GW Group changes are checked iff the interface is not the empty string or the interface in question is not the same as the OpenVPN interface.
(cherry picked from commit 15f8062b42)
OpenVPN instances resync if interface IP change occurs.
At boot, the interface is the empty string, so resync is mandatory to generate OpenVPN files in /var/etc/openvpn.
(cherry picked from commit 7071aab3b2)
- Prevents potential race condition at startup resulting in failure to start OpenVPN instances.
- In cases where interface file is not present the openvpn_resync function handles a restart correctly.
(cherry picked from commit 614ca41e09)
Implementation now checks if OpenVPN client/server running on gateway group should resync when IP changes occur or if cables are unplugged/replugged.
(cherry picked from commit c46d0b12d6)
After discuss with JimP we agreed it would be a better approach than
bdaa5235d4 if we add username tag to the list of tags that use CDATA
(cherry picked from commit ce76d1e41b)
It seems DNSimple started using HTTP/2, which broke the regex the dnsimple updater was using to check for success. I changed it to use the CURLINFO_HTTP_CODE instead.
I noticed several other providers are using the regex status match instead of HTTP_CODE, but I didn't touch those. I haven't written any PHP in 20 years, and I don't use those providers to test them. This change works for me for DNSimple now.
The HTTP/2 response the API was sending back (missing the "OK" the regex was looking for):
```
HTTP/2 200 \r\n
server: nginx
...
```
Fixes Ticket #9580
(cherry picked from commit 6c7d0571d5)
Fixed wildcard variable not being set correctly.
Updated CURLOPT_URL according to provider's documentation.
Added support for MX records.
(cherry picked from commit cedc818460)
* Move code to function to avoid unnecessary duplication of code
* Clean up the logic to avoid further redundancies
* Set keyid type to be quoted and to have its type prefixed
(cherry picked from commit 3a73fc74ca)
Most obvious problem was when manually switching from WANGW1 to WANGW2 it showed both as (default) after saving the setting and before applying it. Also after applying it would require another page refresh to show the updated situation. Also add a little information box that shows what (Default) means for the user.
(cherry picked from commit 3cd21b4e4c)
Allow ACB to be suppressed by including magic string in the backup description
Transmit max number of manual backups in the ACB
(cherry picked from commit 6f6299a3a6)
