security: hide sensitive auth_password in kolla_container module logs
Added no_log=True to the 'common_options' argument in generate_module() to prevent the auth_password and other sensitive data from being printed in Ansible logs during container operations. This improves security by hiding credentials from logs. This change addresses the issue where auth_password was visible in logs despite already having no_log=True on the auth_password parameter itself, because it was nested inside common_options dict without no_log protection. Closes-Bug: #2120302 Change-Id: I2064f822bda1c2618605ecfb9bf26ad820ccbbf2 Signed-off-by: Piotr Milewski <vurmil@gmail.com>
This commit is contained in:
committed by
Michal Nasiadka
parent
f5d6bf6e61
commit
9e8b2c4661
@@ -268,7 +268,8 @@ def generate_module():
|
|||||||
# NOTE(r-krcek): arguments_spec should also be reflected in the list of
|
# NOTE(r-krcek): arguments_spec should also be reflected in the list of
|
||||||
# arguments in service-check-containers role
|
# arguments in service-check-containers role
|
||||||
argument_spec = dict(
|
argument_spec = dict(
|
||||||
common_options=dict(required=False, type='dict', default=dict()),
|
common_options=dict(required=False, type='dict',
|
||||||
|
default=dict(), no_log=True),
|
||||||
action=dict(required=True, type='str',
|
action=dict(required=True, type='str',
|
||||||
choices=['compare_container',
|
choices=['compare_container',
|
||||||
'compare_image',
|
'compare_image',
|
||||||
|
|||||||
13
releasenotes/notes/bug-2120302-824ede145936a6eb.yaml
Normal file
13
releasenotes/notes/bug-2120302-824ede145936a6eb.yaml
Normal file
@@ -0,0 +1,13 @@
|
|||||||
|
---
|
||||||
|
security:
|
||||||
|
- |
|
||||||
|
Added no_log=True to the ``common_options`` argument in generate_module()
|
||||||
|
to prevent the auth_password and other sensitive data from being printed
|
||||||
|
in Ansible logs during container operations when
|
||||||
|
``docker_registry_password`` was set. This improves security by hiding
|
||||||
|
credentials from logs.
|
||||||
|
|
||||||
|
This change addresses the issue where auth_password was visible in log
|
||||||
|
despite already having no_log=True on the auth_password parameter itself,
|
||||||
|
because it was nested inside common_options dict without no_log protection.
|
||||||
|
`LP#2120302 <https://launchpad.net/bugs/2120302>`__
|
||||||
@@ -36,7 +36,8 @@ class ModuleArgsTest(base.BaseTestCase):
|
|||||||
|
|
||||||
def test_module_args(self):
|
def test_module_args(self):
|
||||||
argument_spec = dict(
|
argument_spec = dict(
|
||||||
common_options=dict(required=False, type='dict', default=dict()),
|
common_options=dict(required=False, type='dict', default=dict(),
|
||||||
|
no_log=True),
|
||||||
action=dict(
|
action=dict(
|
||||||
required=True, type='str',
|
required=True, type='str',
|
||||||
choices=['compare_container',
|
choices=['compare_container',
|
||||||
|
|||||||
Reference in New Issue
Block a user