(superset) branch chore/import-validation-debug-log created (now 0155015347d)

[ru...@apache.org]

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

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


      at 0155015347d chore(importers): log field names instead of full config on validation failure

This branch includes the following new commits:

     new 0155015347d chore(importers): log field names instead of full config on validation failure

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.

(superset) 01/01: chore(importers): log field names instead of full config on validation failure

[ru...@apache.org]

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 <ev...@rusackas.com>
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 <no...@anthropic.com>
---
 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)