Skip to content

Commit 7c4ebdd

Browse files
committed
Fix security issues flagged in PR #593 code scanning.
Signed-off-by: lakshmj <[email protected]>
1 parent a59e38d commit 7c4ebdd

4 files changed

Lines changed: 7 additions & 7 deletions

File tree

plugins/modules/nslaslicense_offline.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -43,14 +43,12 @@
4343
- Can also be set via the C(NETSCALER_NITRO_USER) environment variable.
4444
type: str
4545
required: true
46-
no_log: true
4746
nitro_pass:
4847
description:
4948
- The password for the NetScaler ADC appliance.
5049
- Can also be set via the C(NETSCALER_NITRO_PASS) environment variable.
5150
type: str
5251
required: true
53-
no_log: true
5452
nitro_protocol:
5553
description:
5654
- Protocol used to communicate with the NITRO API.
@@ -148,7 +146,6 @@
148146
las_secrets_json: /etc/netscaler/zmcd_secrets.json
149147
150148
"""
151-
152149
RETURN = r"""
153150
---
154151
changed:
@@ -213,7 +210,7 @@ def main():
213210
request_pem=dict(required=True, type="str"),
214211
request_ed=dict(required=True, type="str", choices=["Advanced", "Premium", "Standard"]),
215212
is_fips=dict(type="bool", default=False),
216-
las_secrets_json=dict(required=True, type="str"),
213+
las_secrets_json=dict(required=True, type="str", no_log=False),
217214
host_key_checking=dict(type="bool", default=True),
218215
)
219216

tests/sanity/ignore-2.15.txt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -952,4 +952,5 @@ plugins/modules/ospf6interface.py validate-modules:missing-gplv3-license # We us
952952
plugins/modules/ospf6router.py validate-modules:missing-gplv3-license # We use MIT license
953953
plugins/modules/ospfrouter.py validate-modules:missing-gplv3-license # We use MIT license
954954
plugins/modules/ospfinterface.py validate-modules:missing-gplv3-license # We use MIT license
955-
plugins/modules/bgprouter.py validate-modules:missing-gplv3-license # We use MIT license
955+
plugins/modules/bgprouter.py validate-modules:missing-gplv3-license # We use MIT license
956+
plugins/modules/nslaslicense_offline.py validate-modules:missing-gplv3-license # We use MIT license

tests/sanity/ignore-2.16.txt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -952,4 +952,5 @@ plugins/modules/ospf6interface.py validate-modules:missing-gplv3-license # We us
952952
plugins/modules/ospf6router.py validate-modules:missing-gplv3-license # We use MIT license
953953
plugins/modules/ospfrouter.py validate-modules:missing-gplv3-license # We use MIT license
954954
plugins/modules/ospfinterface.py validate-modules:missing-gplv3-license # We use MIT license
955-
plugins/modules/bgprouter.py validate-modules:missing-gplv3-license # We use MIT license
955+
plugins/modules/bgprouter.py validate-modules:missing-gplv3-license # We use MIT license
956+
plugins/modules/nslaslicense_offline.py validate-modules:missing-gplv3-license # We use MIT license

tests/sanity/ignore-2.17.txt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -952,4 +952,5 @@ plugins/modules/ospf6interface.py validate-modules:missing-gplv3-license # We us
952952
plugins/modules/ospf6router.py validate-modules:missing-gplv3-license # We use MIT license
953953
plugins/modules/ospfrouter.py validate-modules:missing-gplv3-license # We use MIT license
954954
plugins/modules/ospfinterface.py validate-modules:missing-gplv3-license # We use MIT license
955-
plugins/modules/bgprouter.py validate-modules:missing-gplv3-license # We use MIT license
955+
plugins/modules/bgprouter.py validate-modules:missing-gplv3-license # We use MIT license
956+
plugins/modules/nslaslicense_offline.py validate-modules:missing-gplv3-license # We use MIT license

0 commit comments

Comments
 (0)