fix fix should_store_payload_and_result for ckan 2.10

DataShades · Nov 25, 2024 · 900cb95 · 900cb95
1 parent 415d8c5
commit 900cb95
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/ckanext/event_audit/config.py b/ckanext/event_audit/config.py
@@ -22,6 +22,7 @@
 CONF_API_TRACK_ENABLED = "ckanext.event_audit.track_api"
 
 CONF_STORE_PAYLOAD_AND_RESULT = "ckanext.event_audit.store_payload_and_result"
+DEF_STORE_PAYLOAD_AND_RESULT = False
 
 CONF_BATCH_SIZE = "ckanext.event_audit.batch.size"
 CONF_BATCH_TIMEOUT = "ckanext.event_audit.batch.timeout"
@@ -82,7 +83,7 @@ def is_api_log_enabled() -> bool:
 def should_store_payload_and_result() -> bool:
     """Check if the payload and result should be stored in the event. Works
     only for in-built listeners."""
-    return tk.config[CONF_STORE_PAYLOAD_AND_RESULT]
+    return tk.config.get(CONF_STORE_PAYLOAD_AND_RESULT, DEF_STORE_PAYLOAD_AND_RESULT)
 
 
 def get_batch_size() -> int:

diff --git a/ckanext/event_audit/config_declaration.yaml b/ckanext/event_audit/config_declaration.yaml
@@ -71,6 +71,7 @@ groups:
         description: A list of models to track
         type: list
         editable: true
+        default: ""
 
       - key: ckanext.event_audit.store_payload_and_result
         description: Store the payload and result of the event