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
_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
