[PATCH 0/2] Add sensitive-info.log and use it in the Geoloc module

Vratislav Podzimek vpodzime at redhat.com
Wed Aug 28 08:27:57 UTC 2013


On Tue, 2013-08-27 at 17:11 +0200, Martin Kolman wrote:
> The first patch adds a new logger for sensitive information, that should not be
> present in the usual log files that are both stored on the installed system and
> routinely uploaded to Bugzilla.
> But such information might still be useful for debugging, so it can be stored
> in the sensitive-info.log file, that is only present during the installation
> and will not be copied to the installed system.
> 
> Like this, the user can decide to provided the sensitive debugging information
> (after possibly sanitizing it) but when no action is taken, all the sensitive
> data in the log is destroyed once installation ends.
> 
> The second patch makes the Geoloc module use the sensitive info logger for
> logging sensitive location specific information to the sensitive info log file.
> 
> Martin Kolman (2):
>   Add new logger for sensitive information
>   Use the sensitive-info log for sensitive location info
> 
>  pyanaconda/anaconda_log.py | 9 +++++++++
>  pyanaconda/geoloc.py       | 6 ++++--
>  2 files changed, 13 insertions(+), 2 deletions(-)
These both look good to me. If everybody is okay with this resolution
for the issue (expressed as not pointing out what the problem might
be), I'm gonna git-am this patch tomorrow, as Martin is on a PTO.

-- 
Vratislav Podzimek

Anaconda Rider | Red Hat, Inc. | Brno - Czech Republic



More information about the anaconda-patches mailing list