Feature: Zabbix service widget

[ping-localhost]

Proposed change

The user will be able to add the widget to the services.yaml file using the following configuration:

widget:
  type: zabbix
  url: http://zabbix.host.or.ip/zabbix
  key: your-api-key

Relevant feature request: https://github.com/gethomepage/homepage/discussions/1953

Notes

I did somewhat break the Service Widget Guidelines.

• Zabbix exposes 6 alert fields by default, but I can cut it down to just 4.
• Zabbix requires an auth-field in the JSON RPC call for API tokens, which I could not figure out how to do with the generic jsonrpc proxy.

Let me know what I should change to have a valid pull request.

Type of change

• [x] New service widget
• [ ] Bug fix (non-breaking change which fixes an issue)
• [ ] New feature (non-breaking change which adds functionality)
• [ ] Documentation only
• [ ] Other (please explain)

Checklist:

• [x] If applicable, I have added corresponding documentation changes.
• [ ] If applicable, I have reviewed the feature and / or service widget guidelines.
• [x] I have checked that all code style checks pass using pre-commit hooks and linting checks.
• [x] If applicable, I have tested my code for new features & regressions on both mobile & desktop devices, using the latest version of major browsers.

[ping-localhost]

Well, you need to change both of those things, that's why we have the guidelines.

I've removed the two lowest priority fields for now.

Im certain the existing jsonrpc proxy can be modified to accept body params, or just simply adding logic to use widget.key if its supplied instead of username/pw.

I've tried to do that, by passing the params via the URL itself and then parsing it again within the proxy. Hopefully this is okay.

We now also pass the actual widget object to the sendJsonRpcRequest so we can decide if we need to use username + password or the key.

Setting up a Zabbix instance to test does not seem trivial

If you're using Proxmox, you could use tteck's proxmox scripts to setup a Zabbix LXC.

[ping-localhost]

As long as Authorization stays capitalized, I'm happy with the changes in 4a0b18b. Zabbix doesn't accept the API token if the header is authorization.

[ping-localhost]

Perfect, thank you for fixing my mistake. I'll make sure to do better on the next widget 😁

[adm2k]

So is this widget usable at this time? I set it up but getting this:

[ping-localhost]

So is this widget usable at this time? I set it up but getting this:

There is no new release yet, so it is not available yet.

[adm2k]

Ok, thanks for letting me know...

[ping-localhost]

@adm2k FYI, the widget is now available

[adm2k]

Thanks, working now...