Skip to content

Commit b192863

Browse files
committed
fix(payments): suppress codeql clear-text-logging false positive
The validate JSON output logs result.error.message, which CodeQL flags because the upstream validation builds error strings from env-var NAMES (e.g. AGENTCORE_CREDENTIAL_FOO_API_KEY_SECRET — the name of the env var, not its value). The names are deterministic public strings derived from the credential name; they never contain credential values. Suppress the alert with a justification.
1 parent bc77650 commit b192863

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

src/cli/commands/validate/command.tsx

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,10 @@ export const registerValidate = (program: Command) => {
1616
if (result.success) {
1717
console.log(JSON.stringify({ success: true }));
1818
} else {
19+
// codeql[js/clear-text-logging]: result.error.message contains validation messages
20+
// and env-var NAMES (e.g. "AGENTCORE_CREDENTIAL_FOO_API_KEY_SECRET" — the variable
21+
// name, not its value). It never contains credential values; the validate flow only
22+
// checks .env.local presence and references vars by name.
1923
console.log(JSON.stringify({ success: false, error: result.error.message }));
2024
}
2125
process.exit(result.success ? 0 : 1);

0 commit comments

Comments
 (0)