Merge pull request #709 from EvanBldy/master

[files] Add default cache-control header to every static files + allo…

zou/app/blueprints/comments/resources.py

@@ -14,6 +14,7 @@
     tasks_service,
     user_service,
 )
+from zou.app import config
 
 
 class DownloadAttachmentResource(Resource):
@@ -64,6 +65,7 @@ def get(self, attachment_file_id, file_name):
                 mimetype=attachment_file["mimetype"],
                 as_attachment=False,
                 download_name=attachment_file["name"],
+                max_age=config.CLIENT_CACHE_MAX_AGE,
             )
         except Exception:
             abort(404)

zou/app/blueprints/files/resources.py

@@ -32,7 +32,9 @@
 )
 
 
-def send_storage_file(working_file_id, as_attachment=False):
+def send_storage_file(
+    working_file_id, as_attachment=False, max_age=config.CLIENT_CACHE_MAX_AGE
+):
     """
     Send file from storage. If it's not a local storage, cache the file in
     a temporary folder before sending it. It accepts conditional headers.
@@ -58,6 +60,7 @@ def send_storage_file(working_file_id, as_attachment=False):
             mimetype=mimetype,
             as_attachment=as_attachment,
             download_name=download_name,
+            max_age=max_age,
         )
     except IOError as e:
         current_app.logger.error(e)

zou/app/blueprints/previews/resources.py

@@ -127,7 +127,7 @@ def send_storage_file(
     extension,
     mimetype="application/octet-stream",
     as_attachment=False,
-    max_age=60 * 60 * 24 * 7
+    max_age=config.CLIENT_CACHE_MAX_AGE,
 ):
     """
     Send file from storage. If it's not a local storage, cache the file in

zou/app/config.py

@@ -36,6 +36,8 @@
 JWT_COOKIE_SAMESITE = "Lax"
 JWT_IDENTITY_CLAIM = "sub"
 
+CLIENT_CACHE_MAX_AGE = int(os.getenv("CLIENT_CACHE_MAX_AGE", 604800))
+
 DATABASE = {
     "drivername": os.getenv("DB_DRIVER", "postgresql+psycopg"),
     "host": os.getenv("DB_HOST", "localhost"),