Skip to content

CSI-5997#835

Open
uriziv1 wants to merge 1 commit intorelease-1.13.1from
csi-5997-pr
Open

CSI-5997#835
uriziv1 wants to merge 1 commit intorelease-1.13.1from
csi-5997-pr

Conversation

@uriziv1
Copy link
Copy Markdown
Member

@uriziv1 uriziv1 commented Jan 29, 2026

Since CSI-5997, node_id contains only node_name.
Hence, we only validate type and len.
"""
logger.debug("validating node id: %s", node_id)

Check failure

Code scanning / CodeQL

Clear-text logging of sensitive information High

This expression logs
sensitive data (password)
as clear text.
This expression logs
sensitive data (secret)
as clear text.
This expression logs
sensitive data (secret)
as clear text.
This expression logs
sensitive data (secret)
as clear text.
This expression logs
sensitive data (secret)
as clear text.
This expression logs
sensitive data (secret)
as clear text.
This expression logs
sensitive data (secret)
as clear text.
This expression logs
sensitive data (secret)
as clear text.
This expression logs
sensitive data (secret)
as clear text.
This expression logs
sensitive data (secret)
as clear text.
This expression logs
sensitive data (secret)
as clear text.
This expression logs
sensitive data (secret)
as clear text.
This expression logs
sensitive data (password)
as clear text.
This expression logs
sensitive data (secret)
as clear text.
@shlomitn shlomitn force-pushed the release-1.13.1 branch 5 times, most recently from 8a5fa1f to 222840f Compare March 25, 2026 16:46
@shlomitn shlomitn force-pushed the release-1.13.1 branch 5 times, most recently from aa1a5a2 to c59c90a Compare April 14, 2026 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants