From 37d38249bb73e594fbd2bb48744e0b8d3d141d3d Mon Sep 17 00:00:00 2001
From: Nicolai Buchwitz <nicolai.buchwitz@enda.eu>
Date: Wed, 1 Apr 2020 11:11:25 +0200
Subject: [PATCH] python 3.5 compatibility

---
 check_jitsi.py | 22 ++++++++++++----------
 1 file changed, 12 insertions(+), 10 deletions(-)

diff --git a/check_jitsi.py b/check_jitsi.py
index 69d38a0..4f9de6a 100755
--- a/check_jitsi.py
+++ b/check_jitsi.py
@@ -25,12 +25,13 @@
 # ------------------------------------------------------------------------------
 
 
-import sys
-import re
-import requests
 import argparse
+import re
+import sys
 from enum import Enum
 
+import requests
+
 
 class CheckState(Enum):
     OK = 0
@@ -99,7 +100,7 @@ def __init__(self):
                               'jitter_aggregate', 'total_no_payload_channels', 'total_no_transport_channels']
 
         self.parse_args()
-        self._baseurl = f"http://{self.args.hostname}:{self.args.port}"
+        self._baseurl = "http://{}:{}".format(self.args.hostname, self.args.port)
 
         self._state = CheckState.OK
 
@@ -163,7 +164,8 @@ def check_result(self, rc, message, metrics=None):
 
         if metrics:
             perfdata = '|'
-            perfdata += ' '.join([f"{k}={int(v)}" if type(v) == bool else f"{k}={v}" for k, v in metrics.items()])
+            perfdata += ' '.join(
+                ["{}={}".format(k, int(v)) if type(v) == bool else "{}={}".format(k, v) for k, v in metrics.items()])
 
         print(message, perfdata)
         sys.exit(rc.value)
@@ -171,7 +173,7 @@ def check_result(self, rc, message, metrics=None):
     def _fetch(self, uri):
         try:
             response = requests.get(
-                f"{self._baseurl}{uri}",
+                "{}{}".format(self._baseurl, uri),
             )
         except requests.exceptions.ConnectTimeout:
             self.check_result(CheckState.UNKNOWN, "Could not connect to JVB API: Connection timeout")
@@ -189,14 +191,14 @@ def check_health(self):
         msg = "Jitsi videobridge is healthy"
 
         if not healthy:
-            msg = f"Jitsi videobridge health check failed ({r.status_code})"
+            msg = "Jitsi videobridge health check failed ({})".format(r.status_code)
             self._state = CheckState.CRITICAL
 
         self.check_result(self._state, msg, {})
 
     def _check_simple(self, name):
         value = self.statistics.get(name, 0)
-        msg = f"{value} {name}"
+        msg = "{} {}".format(value, name)
 
         if self._criticial.check(value):
             self._state = CheckState.CRITICAL
@@ -204,12 +206,12 @@ def _check_simple(self, name):
             self._state = CheckState.WARNING
 
         self.check_result(self._state, msg,
-                          {name: f"{value};;{self.args.threshold_warning};{self.args.threshold_critical}"})
+                          {name: "{};;{};{}".format(value, self.args.threshold_warning, self.args.threshold_critical)})
 
     def check(self):
         if self.args.mode == 'health':
             self.check_health()
-        if self.args.mode in self._simple_modes:
+        elif self.args.mode in self._simple_modes:
             self._check_simple(self.args.mode)