From 9d5c0d96bd2d29b2487c82c4847fa4a6724a7902 Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Sun, 22 Aug 2021 07:48:40 +0200
Subject: [PATCH] Change Redis cache config: infinite default TTL

---
 app/__init__.py         | 5 +++--
 app/scodoc/sco_cache.py | 5 +++--
 app/scodoc/sco_excel.py | 6 ++++--
 3 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/app/__init__.py b/app/__init__.py
index 5c403923a..001bbc241 100644
--- a/app/__init__.py
+++ b/app/__init__.py
@@ -35,8 +35,9 @@ moment = Moment()
 
 cache = Cache(  # XXX TODO: configuration file
     config={
-        # "CACHE_TYPE": "MemcachedCache"
-        "CACHE_TYPE": "RedisCache"
+        # see https://flask-caching.readthedocs.io/en/latest/index.html#configuring-flask-caching
+        "CACHE_TYPE": "RedisCache",
+        "CACHE_DEFAULT_TIMEOUT": 0,  # by default, never expire
     }
 )
 
diff --git a/app/scodoc/sco_cache.py b/app/scodoc/sco_cache.py
index 065f51a10..c107fb33b 100644
--- a/app/scodoc/sco_cache.py
+++ b/app/scodoc/sco_cache.py
@@ -232,9 +232,10 @@ class NotesTableCache(ScoDocCache):
 
         t0 = time.time()
         nt = notes_table.NotesTable(formsemestre_id)
-        dt = time.time() - t0
-        log("caching formsemestre_id=%s (%gs)" % (formsemestre_id, dt))
+        t1 = time.time()
         _ = cls.set(formsemestre_id, nt)
+        t2 = time.time()
+        log(f"cached formsemestre_id={formsemestre_id} ({(t1-t0):g}s +{(t2-t1):g}s)")
         g.nt_cache[formsemestre_id] = nt
         return nt
 
diff --git a/app/scodoc/sco_excel.py b/app/scodoc/sco_excel.py
index f2b1d9b0a..d04a1fb61 100644
--- a/app/scodoc/sco_excel.py
+++ b/app/scodoc/sco_excel.py
@@ -228,8 +228,10 @@ class ScoExcelSheet:
          un workbook est crée et associé à cette feuille.
         """
         # Le nom de la feuille ne peut faire plus de 31 caractères.
-        # On  abrège le nom pour les cas connus de dépassement puis si la taille est toujours > 31 on tronque (on pourrait remplacer par 'feuille' ?)
-        self.sheet_name = sheet_name[:31]  # if len(sheet_name) > 31: sheet_name = 'Feuille' ?
+        # si la taille du nom de feuille est > 31 on tronque (on pourrait remplacer par 'feuille' ?)
+        self.sheet_name = sheet_name[
+            :31
+        ]  # if len(sheet_name) > 31: sheet_name = 'Feuille' ?
         self.rows = []  # list of list of cells
         # self.cells_styles_lico = {}  # { (li,co) : style }
         # self.cells_styles_li = {}  # { li : style }