Add support for Zabbix 5.x

[root-expert]

Pull Request (PR) description

Adds support for Zabbix 5.x.
Maybe more tests are needed
Depends on #742

This Pull Request (PR) fixes the following issues

Fixes #731
Closes #737
Closes #736
Closes #721
Closes #715
Closes #694

[DJClean]

As I was looking into upgrading to Zabbix 5.0 myself, looked into this PR

But it seems to break on web.pp because of the custom_fragment with all the php_value settings
The var $apache_vhost_custom_fragment is being set, but not used to set the fragment in the vhost if I saw it right. Overall it seems to work just fine if I remove the fragment manually

[root-expert]

Thanks! I'll look into it and report back

[DJClean]

Already pushed a PR to your branch with the fix :)

[jameskirsop]

Could we also consider rolling in my branch for adding support for the SAML / SSO settings introduced in 5.x? Doing this will allow #693 to be closed.

If it's deemed not appropriate I'll file a seperate PR.

[jadestorm]

Not to pile on =) But this setting is pretty important for Zabbix 5:
#734
(it has a PR already)

[jadestorm]

PS aside from that I am using @root-expert 's PR fork directly right now for my 5.2 install and it's working great! Mind you I'm not (yet) using the exported resources functionality.

[root-expert]

Could we also consider rolling in my branch for adding support for the SAML / SSO settings introduced in 5.x? Doing this will allow #693 to be closed.

If it's deemed not appropriate I'll file a seperate PR.

I think you should open a new request after rebasing with my branch. Also you should write some specs ;)

Not to pile on =) But this setting is pretty important for Zabbix 5:
#734
(it has a PR already)

I believe that PR could be merged if rebased as well.