Skip to content

Potential fix for code scanning alert no. 2: Clear-text logging of sensitive information#30

Merged
wobrschalek merged 1 commit intomainfrom
alert-autofix-2
Feb 9, 2026
Merged

Potential fix for code scanning alert no. 2: Clear-text logging of sensitive information#30
wobrschalek merged 1 commit intomainfrom
alert-autofix-2

Conversation

@wobrschalek
Copy link
Contributor

Potential fix for https://github.com/Open-MBEE/flexo_syside/security/code-scanning/2

In general, the problem should be fixed by ensuring that no part of the API key (or any other secret) is written to logs, even partially. Instead of logging a prefix of the token, you can log only non-sensitive metadata, such as whether a token is present, its length, or that authentication has been configured, without including any actual characters from the key itself.

The best targeted fix here is to modify the verbose logging line at line 128 so that it does not include api_key[:10] or any other substring of the secret. We can keep the informational value by logging a generic message like “[Flexo] Authentication token configured.” or, if needed, the length of the token (which is usually not sensitive), e.g. len(api_key) without printing characters of the secret. This change is confined to commit_sysml_to_flexo in src/flexo_syside_lib/committer.py and does not alter any functional behavior related to authentication or API calls; it only changes what gets printed when verbose is True. No additional imports, methods, or definitions are required.

Suggested fixes powered by Copilot Autofix. Review carefully before merging.

…nsitive information

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@wobrschalek wobrschalek marked this pull request as ready for review February 9, 2026 23:24
@wobrschalek wobrschalek merged commit 5464ab1 into main Feb 9, 2026
4 of 5 checks passed
@wobrschalek wobrschalek deleted the alert-autofix-2 branch February 9, 2026 23:26
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.

1 participant