[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

[PATCH] Another attempt at making the attrSkipList work.



In the most local frame, the name of whatever object we're dumping isn't
what the attrSkipList is, it's "obj".  Therefore the attrSkipList needs
to not start with the object's name, and dump needs to put "obj" on the
front of each attribute.
---
 src/__init__.py |    8 +++++++-
 src/dump.py     |    2 +-
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/__init__.py b/src/__init__.py
index ebd5a92..185a528 100644
--- a/src/__init__.py
+++ b/src/__init__.py
@@ -47,7 +47,13 @@ class Config(object):
                              any attributes found with the same name as an
                              element of this list will not be written to
                              the dump.  This is to prevent writing potentially
-                             sensitive information like passwords.
+                             sensitive information like passwords.  The names
+                             must be given without the leading name of the
+                             object passed to handler.install().  For instance,
+                             if handler.install() gets an Anaconda instance
+                             with the name "anaconda" and you want to skip
+                             anaconda.id.rootPassword, "id.rootPassword" should
+                             be listed in attrSkipList.
            bugFiler       -- An AbstractFiler instance.  This is required for
                              saving to remote bug tracking systems.  Of course
                              this must be an instance of a real subclass, not
diff --git a/src/dump.py b/src/dump.py
index 3ace433..3da2091 100644
--- a/src/dump.py
+++ b/src/dump.py
@@ -201,7 +201,7 @@ class ExceptionDump(object):
         # and ignore them.
         for k in self.conf.attrSkipList:
             try:
-                eval("idSkipList.append(id(%s))" % k, None, localVars)
+                eval("idSkipList.append(id(obj.%s))" % k, None, localVars)
             except:
                 pass
 
-- 
1.6.5.1


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]