This is an automated email from the ASF dual-hosted git repository.

rusackas pushed a commit to branch chore/import-validation-debug-log
in repository https://gitbox.apache.org/repos/asf/superset.git

commit 0155015347dd9842d5e1ddd775306fbf55628d5f
Author: Evan Rusackas <[email protected]>
AuthorDate: Fri Jul 24 13:40:17 2026 -0700

    chore(importers): log field names instead of full config on validation 
failure
    
    The debug log dumped the entire parsed config dict when schema
    validation failed. Those blobs can be enormous (inline example data,
    multi-line keys) and bury the actual validation error logged just
    above. Log the file name and its top-level field names instead.
    
    Co-Authored-By: Claude Fable 5 <[email protected]>
---
 superset/commands/importers/v1/utils.py | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/superset/commands/importers/v1/utils.py 
b/superset/commands/importers/v1/utils.py
index f2fe9eaba0a..a0829de762c 100644
--- a/superset/commands/importers/v1/utils.py
+++ b/superset/commands/importers/v1/utils.py
@@ -230,7 +230,11 @@ def load_configs(
                     prefix,
                     exc.messages,
                 )
-                logger.debug("Config content that failed validation: %s", 
config)
+                # Log field names only; full values can be huge (e.g. inline
+                # example data) and drown out the validation error above.
+                logger.debug(
+                    "Config fields present in %s: %s", file_name, 
sorted(config)
+                )
                 exc.messages = {file_name: exc.messages}
                 exceptions.append(exc)
 

Reply via email to