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
run_proces logs the command without escaping parmaeters such that the coammns logged are not valid #73321
Comments
So, the arguments to run_process are not escaped when logged such that the logged command is un-parsable. self.run_process(['notify-send', '--app-name', 'Mozilla Build System', 'Mozilla Build System', msg]) where msg='Build complete' ends up logging the following: /usr/bin/notify-send --app-name Mozilla Build System Mozilla Build System Build complete Where to be a valid command it needs to be: /usr/bin/notify-send --app-name 'Mozilla Build System' 'Mozilla Build System' 'Build complete' So, I think this needs to either not log the command at all or for each parameter replace any occurrence of the character "'" with "\'" and then enclose the entire parameter with "'" to make sure the logged command can actually be properly parsed. |
What is run_process? I'm not getting any hits from grep on the standard library. |
well i could be screwed up perhaps it is something provided in the mozilla On Mon, Jan 2, 2017 at 4:02 PM, R. David Murray <report@bugs.python.org>
|
OK i t appears it might be a third party python utility library. I will On Mon, Jan 2, 2017 at 4:07 PM, William Gianopoulos <report@bugs.python.org>
|
I would like to keep this open until I figure this out so I can provide a pointer to where the real issue is being tracked. I should have time to do that tomorrow. SOunds like this is part of some third-party add-on python library that is normally provided with linux builds. |
It seems it is part of the Mozilla build system. I closed this issue. |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: