.gitignore

@@ -1 +0,0 @@
-__pycache__/

README.md

@@ -19,8 +19,6 @@ You need Python 3 and [Requests](http://python-requests.org/). You can install R
 
 Then, copy the script `check_expire` and the file `ianardap.py`to the directory of local plugins.
 
-The use of a cache directory is not mandatory, but highly recommended. It can be either `$XDG_CACHE_HOME/ianardap` or `$HOME/.ianardapcaches` depending on the environment variables set.
-
 ## Icinga configuration
 
 If you use Icinga, here is a possible definition of the command:
@@ -37,7 +35,6 @@ object CheckCommand "expiration" {
       "-v" = { set_if = "$expiration_verbose$" }
   }
 
-  env.XDG_CACHE_HOME = "/var/cache/nagios"
 }
 
 apply Service "expiration" {
@@ -63,13 +60,6 @@ object Host "bortzmeyer-org" {
 }
 ```
 
-If needed, create the cache directory accordingly:
-
-```
-mkdir /var/cache/nagios
-chown nagios: /var/cache/nagios
-```
-
 ## Zabbix configuration
 
 For monitoring systems that do not rely on exit codes but on calculation mechanism based on the metric they receive you can use the `-u` option that will only return the expiration date in unixtime format.

check_expire

@@ -53,8 +53,6 @@ def usage(msg=None):
         print("")
 
 def details():
-    if database and not database.cachefile:
-        print(" (WARNING: no cache directory used, please set environement variable XDG_CACHE_HOME or HOME.)", end="")
     if verbose:
         print(" RDAP database \"%s\", version %s published on %s, retrieved on %s, RDAP server is %s" % \
               (database.description, database.version, database.publication, database.retrieved, server))

ianardap.py

@@ -21,8 +21,7 @@ IANABASES = {"domains": "https://data.iana.org/rdap/dns.json",
              "v6prefixes": "https://data.iana.org/rdap/ipv6.json",
              "as": "https://data.iana.org/rdap/asn.json",
              "objects": "https://data.iana.org/rdap/object-tags.json"}
-CACHE = os.environ["XDG_CACHE_HOME"] + "/ianardap" if "XDG_CACHE_HOME" in os.environ else \
+CACHE = os.environ["HOME"] + "/.ianardapcaches" 
-        (os.environ["HOME"] + "/.ianardapcaches" if "HOME" in os.environ else None)
 MAXAGE = 24 # Hours. Used only if the server no longer gives the information.
 IANATIMEOUT = 10 # Seconds
 MAXTESTS = 3 # Maximum attempts to get the database
@@ -61,33 +60,28 @@ file (see the documentation of the module).
         """
         
         cache_valid = False
+        if not os.path.exists(cachedir):
+            os.mkdir(cachedir)
         self.category = category
-        self.cachefile = None
+        cachefile = os.path.join(cachedir, category)
-        self.lockname = None
+        if pickleformat:
-        self.expirationfile = None
+            self.cachefile = cachefile + ".pickle"
-        if cachedir:
+        else:
-            if not os.path.exists(cachedir):
+            self.cachefile = cachefile + ".json"
-                os.mkdir(cachedir)
+        self.lockname = self.cachefile + ".lock"
-            cachefile = os.path.join(cachedir, category)
+        self.expirationfile = self.cachefile + ".expires"
-            if pickleformat:
-                self.cachefile = cachefile + ".pickle"
-            else:
-                self.cachefile = cachefile + ".json"
-            self.lockname = self.cachefile + ".lock"
-            self.expirationfile = self.cachefile + ".expires"
         if maxage is not None:
-            self.expirationtime = time.mktime((datetime.datetime.now() + \
+            with open(self.expirationfile, 'w'):
-                                              datetime.timedelta(hours=maxage)).timetuple())
+                self.expirationtime = time.mktime((datetime.datetime.now() + \
-            if self.expirationfile:
+                                                  datetime.timedelta(hours=maxage)).timetuple())
-                with open(self.expirationfile, 'w'):
+                os.utime(self.expirationfile,
-                    os.utime(self.expirationfile,
+                         times = (self.expirationtime, self.expirationtime))
-                             times = (self.expirationtime, self.expirationtime))
         loaded = False
         tests = 0
         errmsg = "No error"
         while not loaded and tests < MAXTESTS:
             self.lock()
-            if self.cachefile and os.path.exists(self.cachefile) and \
+            if os.path.exists(self.cachefile) and \
                (pathlib.Path(self.expirationfile).exists() and \
                 datetime.datetime.fromtimestamp(os.path.getmtime(self.expirationfile)) > \
                 datetime.datetime.now()):
@@ -145,10 +139,9 @@ file (see the documentation of the module).
                 try:
                     content = response.content
                     database = json.loads(content)
-                    if self.expirationfile:
+                    with open(self.expirationfile, 'w'):
-                        with open(self.expirationfile, 'w'):
+                        os.utime(self.expirationfile,
-                            os.utime(self.expirationfile,
+                                 times = (self.expirationtime, self.expirationtime))
-                                     times = (self.expirationtime, self.expirationtime))
                 except json.decoder.JSONDecodeError:
                     tests += 1
                     errmsg = "Invalid JSON retrieved from %s" % IANABASE
@@ -180,7 +173,7 @@ file (see the documentation of the module).
         else: # IP addresses will be complicated, because of the
               # longest prefix rule.
             raise Exception("Unsupported category %s" % self.category)
-        if self.cachefile and not cache_valid:
+        if not cache_valid:
             self.lock()
             cache = open(self.cachefile, "wb")
             if pickleformat:
@@ -191,14 +184,12 @@ file (see the documentation of the module).
             self.unlock()
         
     def lock(self):
-        if self.lockname:
+        self.lockhandle = open(self.lockname, 'w')
-            self.lockhandle = open(self.lockname, 'w')
+        fcntl.lockf(self.lockhandle, fcntl.LOCK_EX)
-            fcntl.lockf(self.lockhandle, fcntl.LOCK_EX)
 
     def unlock(self):
-        if self.lockname:
+        fcntl.lockf(self.lockhandle, fcntl.LOCK_UN)
-            fcntl.lockf(self.lockhandle, fcntl.LOCK_UN)
+        self.lockhandle.close()
-            self.lockhandle.close()
 
     def find(self, id):
         """Get the RDAP server(s), as an array, for a given identifier. None

test_ianardap.py

@@ -29,9 +29,8 @@ def test_refresh():
     # Force a resfresh
     database = ianardap.IanaRDAPDatabase(maxage=0)
     assert (database.retrieved > (datetime.datetime.now() - datetime.timedelta(minutes=1))) and \
-        ((not database.cachefile) or \
         (datetime.datetime.fromtimestamp(os.path.getmtime(database.cachefile)) > \
-        (datetime.datetime.now() - datetime.timedelta(minutes=1))))
+        (datetime.datetime.now() - datetime.timedelta(minutes=1)))
 
 def test_find_exists():
     database = ianardap.IanaRDAPDatabase()

tests.yaml

@@ -143,14 +143,14 @@ tests:
       retcode: 0
       stderr: ''
 
-    # Iran, expiration date in the past. But down (2024-06-16)
+    # Iran, expiration date in the past
-    #- exe: './check_expire'
+    - exe: './check_expire'
-    #  args:
+      args:
-    #      - '-H'
+          - '-H'
-    #      - 'nic.pars'
+          - 'nic.pars'
-    #  retcode: 2
+      retcode: 2
-    #  partstdout: "already expired"
+      partstdout: "already expired"
-    #  stderr: ''
+      stderr: ''
 
     # Brazil
     - exe: './check_expire'