Skip to content

Commit

Permalink
fix!: handle changed opnsense api since 22.7.x
Browse files Browse the repository at this point in the history
  • Loading branch information
Jörg Dreffein authored and nbuchwitz committed Jul 1, 2024
1 parent f76b277 commit fa90f47
Showing 1 changed file with 10 additions and 6 deletions.
16 changes: 10 additions & 6 deletions check_opnsense.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
# check_opnsense.py - A check plugin for monitoring OPNsense firewalls.
# Copyright (C) 2018 Nicolai Buchwitz <[email protected]>
#
# Version: 0.1.0
# Version: 0.1.1
#
# ------------------------------------------------------------------------------
# This program is free software; you can redistribute it and/or
Expand Down Expand Up @@ -211,15 +211,19 @@ def check_updates(self) -> None:
url = self.get_url("core/firmware/status")
data = self.request(url)

if data["status"] == "ok" and data["status_upgrade_action"] == "all":
count = data["updates"]
if data["status"] == "none":
# no update information available -> trigger check
data = self.request(url, method="post")

has_update = data["status"] in ("update", "upgrade")
needs_reboot = data["status_reboot"] == "1"

if has_update:
self.check_result = CheckState.WARNING
self.check_message = "{} pending updates".format(count)
self.check_message = data["status_msg"]

if data["upgrade_needs_reboot"]:
if needs_reboot:
self.check_result = CheckState.CRITICAL
self.check_message = "{}. Subsequent reboot required.".format(self.check_message)
else:
self.check_message = "System up to date"

Expand Down

0 comments on commit fa90f47

Please sign in to comment.