URL: https://github.com/freeipa/freeipa/pull/231
Title: #231: Do not log DM password in ca/kra installation logs

tomaskrizek commented:
"""
I agree. We need to re-add `self.dm_password` to `nolog_list`, just like it was 
before I removed it 
[here](https://github.com/freeipa/freeipa/commit/9340a1417acf120fed3e9ffbe9d658d3456743a1#diff-36dfe273a301d6b5ea2bbcf89c7cd661L167)

There is no reason to change it. I originally removed the line, because I 
thought I could remove `dm_password` from `DogtagInstance` all together, but 
that turned out not to be the case.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/231#issuecomment-262171995
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to