This doesn't work out at all for the fact that the control picks
up the first instance it finds, either client or server, but not
clearly stating so.  That's why there is a status page with all
the services for clients and servers...

(cherry picked from commit 7b17ac34)

(cherry picked from commit 918df6ea)

(cherry picked from commit d03b53b3)

fix for https://github.com/opnsense/core/issues/296
and  https://github.com/opnsense/core/issues/347

when no servers are selected, all still apply, which should keep current configurations valid.

(cherry picked from commit d101b28e)

(legacy) align help message with openvpn for vpn_openvpn_server.php, closes https://github.com/opnsense/core/issues/774

(cherry picked from commit 8389079e)

(cherry picked from commit 3f2c5304)

(cherry picked from commit d9c91c3f)

(cherry picked from commit 33d8497b)

(cherry picked from commit 63938f7b)

(cherry picked from commit 30f6c2bc)

(legacy) add copy/move to vpn_openvpn_csc.php overview + some js cleanups. closes https://github.com/opnsense/core/issues/442

(cherry picked from commit a0587c03)

(cherry picked from commit 0f2f9ce7)

(cherry picked from commit 9784c8b0)

services.inc already includes unbound.inc and miniupnpd.inc so
it's safe to do this...

(cherry picked from commit f90e96f2)

(cherry picked from commit 79edae2c)

(cherry picked from commit d9810025)

(cherry picked from commit 3600c6a3)

(cherry picked from commit de888244)

(cherry picked from commit 60f0f461)

(cherry picked from commit 6f2daad5)

(cherry picked from commit 4d3ac63a)

(cherry picked from commit 77696bd8)

Improve redirect to avoid removing the savemsg which can be useful
and start to use the service array instead of the plain name.  This
is going to help with moving all the start/restart/stop info into
the service array creation for base services as well as plugin
services.

(cherry picked from commit aa8202aa)

(proxy) set unset description on X-Forwarded-For. closes https://github.com/opnsense/core/issues/790

(cherry picked from commit 8b701717)

(cherry picked from commit df7e8660)

(cherry picked from commit 23b34746)

(cherry picked from commit 731d5ab6)

(cherry picked from commit 75bbd009)

(cherry picked from commit ca10a6f2)

(cherry picked from commit 28cf9dba)

(cherry picked from commit 71bd852f)

(cherry picked from commit 89179ee7)

(cherry picked from commit 45d45596)

(cherry picked from commit c0ec7458)

(legacy) fix html error issue in firewall_aliases_edit.php, https://github.com/opnsense/core/issues/785

(cherry picked from commit bcaaf64c)

(cherry picked from commit 4a8d192b)

Most of the time, there is no return value from the functions,
sometimes $retval is not evaluated, and sometimes it just seems
totally convoluted to do it when 99% of the configuration code
does not do it and works fine wihout it.

It's also not practical, because there is no no notion of a
rollback that would have to ensue in such cases.

(cherry picked from commit 1eb5dae7)

(cherry picked from commit 5a260988)

(cherry picked from commit 93c20e1b)

(cherry picked from commit 61cb3745)