The old script had a lot of unused code in it, either stuff that couldn't be configured from OPNsense or dead code.

Revert "(ids) style fix", https://github.com/opnsense/core/commit/aaa0ddafe1179cebc68146bdc3c30b8b90c564b8 can cause "Allowed memory size of ... bytes exhausted", better to set a sane default max here.

This reverts commit aaa0ddaf.

(firmware) add configd "firmware loader" commands to control the contents of /boot/loader.conf.local

plugins can use this call to register new bootup parameters, new commands added:

configctl firmware loader show
configctl firmware loader rm <parameter>
configctl firmware loader edit <parameter> <value>

(crashreport) PHP Warning:  in_array() expects parameter 2 to be array, null given in /usr/local/www/services_unbound.php on line 176

(crashreport) PHP Warning:  Illegal string offset 'rule' in /usr/local/www/diag_logs_filter.php on line 277

(crashreport) limit output chunks to 5MB, if we can't find </head> within that area it's very unlikely that it exists in the output.

report:
PHP Fatal error:  Allowed memory size of 268435456 bytes exhausted (tried to allocate 35997807 bytes) in /usr/local/www/csrf/csrf-magic.php on line 144

(packet capture) limit the maximum number of output lines, prevent "PHP Fatal error:  Allowed memory size of ..."

collected from crashreport

(crashreport) PHP Warning:  in_array() expects parameter 2 to be array, null given in /usr/local/www/vpn_openvpn_csc.php on line 406

changes in services_unbound_advanced.php

fix translation in firewall_virtual_ip.php

* translation fixes (for example submit button)
* remove some not operators
* make short echo tags look more like a template

Relative redirect is used almost everywhere, but to be honest
it looks safer to make all paths absolute.

PR: https://github.com/opnsense/core/issues/1168

PR: https://github.com/opnsense/core/issues/1168

PR: https://forum.opnsense.org/index.php?topic=3573.0

(openvpn) refactor openvpn_get_cipherlist and openvpn_get_digestlist, fix compatibilty with both --show-ciphers outputs

Suggested by: SimonB

Since we weaved the latter into the former, we can now get rid of
the transition code.  The behaviour doesn't change.

This does:

o Enable/disable all text as opposed to confusing icons of
  checkboxes.  Also mark them as "primary".
o Make the info button a pencil / edit button as we can modify
  data from there.
o Minor cleanups.

Missing:

o Removing the toggle buttons for each rule (the edit button
  is sufficient).  Haven't done this because the rowtoggle
  mechanism is probably better adapted by @adschellevis.
o Maybe move the enable disable all buttons back to the bottom.

There is no need to visually overload the field descriptions with
stop chars like a slash, the help is sufficient in explaining that
multiple networks can be used and the chances are high that most
will only ever use one network here.