Skip to content
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

ToolRunner is leaking the command line #1029

Closed
PavelBansky opened this issue Apr 2, 2024 · 3 comments
Closed

ToolRunner is leaking the command line #1029

PavelBansky opened this issue Apr 2, 2024 · 3 comments

Comments

@PavelBansky
Copy link

ToolRunner is leaking command line arguments into logs on the build machine.
This is not good, because we are passing urls with SAS token to cUrl executed by ToolRunner

EXECVE argc=5 a0="/bin/curl" a1="--location" a2=https://cXXXXXXX0.blob.core.windows.net/miXXXXXXXnt/codeql-cli/2.16.3/linux64.zip?sp=rli&st=2024-03-15T17:37:05Z&se=2024-10-16T01:37:05Z&spr=https&sv=2022-11-02&sr=c&sig=lPyj<REDACTED> a3="--output" a4="/mnt/vss/_work/_temp/coXXXXXX/distribution/linux64.zip"

This happens even if run in silent mode with all outputs redirected to null

const result = await runner.exec({silent: true, outStream: null, errStream: null} as IExecOptions);
@aleksandrlevochkin
Copy link
Contributor

Hi @PavelBansky, thank you for reporting this issue, we'll start working on this soon

@ismayilov-ismayil
Copy link
Contributor

Hi @PavelBansky
Please find the explanation below:

  • The outStream is related to executed application (in this case 'curl') but not task itself
  • If outStream is null it uses default process output handler

image

  • The silent parameter is also related to executed application. It means in case of 'true' the output from application will be hidden

  • The task arguments are written by 'debug' method of the task-lib. So, if you disable System.debug parameter in your pipeline you will not see args in the log.

  • But if you want to hide args in any case, there are several options to do it. here is one of them:

  1. Change your task which accepts input parameter, something like that:
    image

  2. Then add new variables to your pipeline with this name and mark it as secret:

This variable will be masked in the logs
image

image

Copy link

github-actions bot commented Aug 2, 2024

This issue has had no activity in 90 days. Please comment if it is not actually stale

@github-actions github-actions bot added the stale label Aug 2, 2024
@github-actions github-actions bot closed this as completed Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants