-
Notifications
You must be signed in to change notification settings - Fork 63
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Issue 964 #971
base: main
Are you sure you want to change the base?
Conversation
Hello there, thanks for opening your first Pull Request. Someone will review it soon. |
|
||
console.log(paymentIntent, amount, eventOutcome) | ||
console.log(paymentIntent, amount, eventOutcome); |
Check failure
Code scanning / CodeQL
Use of externally-controlled format string High
user-provided value
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix AI about 2 months ago
To fix the problem, we should use a format specifier to safely include the paymentIntent
, amount
, and eventOutcome
values in the console.log
statement. This ensures that any user-provided input is treated as a string and does not interfere with the format of the log message.
- Modify the
console.log
statement on line 162 to use a format string with%s
specifiers. - Pass the
paymentIntent
,amount
, andeventOutcome
values as separate arguments to theconsole.log
function.
-
Copy modified line R162
@@ -161,3 +161,3 @@ | ||
|
||
console.log(paymentIntent, amount, eventOutcome); | ||
console.log('Payment Intent: %s, Amount: %s, Event Outcome: %s', paymentIntent, amount, eventOutcome); | ||
|
Hack pod task complete