summaryrefslogtreecommitdiff
path: root/lib/mixlib/shellout
Commit message (Collapse)AuthorAgeFilesLines
* Support array args on windowslcg/win-array-argsLamont Granquist2019-06-061-2/+42
| | | | | | | | | | | | | | | | | | | | | | Implements correct quoting and escaping of arguments on windows. That means that this works right now: ```ruby filename = "c:\program files" shell_out("dir", filename); ``` So all the defensive coding around quotes-around-filepaths -- which were all actually buggy even when they worked (trailing backslashes would fail) -- are unnecessary and arguments can just be passed in as an array and this code will sort it out. We rely on the existing determination of if metacharacters mean it needs to run under cmd and if it needs the ^ quoting which seems to be well tested, and which all runs after this does. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Fix chefstyle warningsTim Smith2019-04-191-2/+2
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 3.0.1 by Chef Expeditorv3.0.1Chef Expeditor2019-04-191-1/+1
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* update travis/appveyor, drop ruby 2.2 support, test on 2.6lcg/version-bump2Lamont Granquist2019-01-041-1/+1
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Bump version to 2.4.4 by Chef Expeditorv2.4.4Chef Expeditor2018-12-121-1/+1
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Bump version to 2.4.3 by Chef Expeditorv2.4.3Chef Expeditor2018-12-071-1/+1
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Bump version to 2.4.2 by Chef Expeditorv2.4.2Chef Expeditor2018-12-061-1/+1
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Bump version to 2.4.1 by Chef Expeditorv2.4.1Chef Expeditor2018-12-061-1/+1
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Update the github links in commentsTim Smith2018-10-231-1/+1
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Fix EmptyWindowsCommand error on Windows when the command starts with spacesEric VERNIER2018-07-301-4/+3
| | | | Signed-off-by: Eric Vernier <zerikv@gmail.com>
* Merge pull request #164 from chef/new_rubyv2.4.0Tim Smith2018-07-231-1/+1
|\ | | | | Release 2.4.0
| * Release 2.4.0new_rubyTim Smith2018-07-231-1/+1
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #158 from MsysTechnologiesllc/dh/MSYS-847_fix_bare_rescuesTim Smith2018-07-231-2/+2
|\ \ | |/ |/| Fix a bare rescue on Windows
| * [MSYS-847] fix bare rescuedheerajd-msys2018-07-031-2/+2
| | | | | | | | Signed-off-by: dheerajd-msys <dheeraj.dubey@msystechnologies.com>
* | master of chefstyle + fixesLamont Granquist2018-07-061-1/+1
|/ | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* remove hashrocket syntaxlcg/remove-hashrocketsLamont Granquist2018-06-252-9/+9
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* add sensitive featuredheerajd-msys2018-06-152-8/+9
| | | | Signed-off-by: dheerajd-msys <dheeraj.dubey@msystechnologies.com>
* [MSYS-629] Added username,password validation for elavated option (#151)Nimisha Sharad2017-08-091-0/+8
| | | | | | * Added username,password validation for elavated option Signed-off-by: nimisha <nimisha.sharad@clogeny.com>
* Release 2.3.2v2.3.2Bryan McLellan2017-07-211-1/+1
| | | | Signed-off-by: Bryan McLellan <btm@loftninjas.org>
* Fix broken method callBryan McLellan2017-07-211-1/+1
| | | | | | Missed one argument to this method call Signed-off-by: Bryan McLellan <btm@loftninjas.org>
* Release 2.3.1v2.3.1Bryan McLellan2017-07-191-1/+1
| | | | Signed-off-by: Bryan McLellan <btm@chef.io>
* Make Mixlib::ShellOut::EmptyWindowsCommand inherit from ShellCommandFailedBryan McLellan2017-07-191-1/+1
| | | | | | | | | Chef rescues Mixlib::ShellOut::ShellCommandFailed, SystemCallError in a few places. Recent refactors caused EmptyWindowsCommand to be correctly raised where we still unintentionally raised SystemCallError due to a bad FFI call. This gets us closer to not needing to cover these edge cases in rescue clauses everywhere. Signed-off-by: Bryan McLellan <btm@chef.io>
* Release 2.3.0v2.3.0Bryan McLellan2017-07-191-1/+1
|
* Update error messages to indicate user who lacks privilege on WindowsBryan McLellan2017-07-191-140/+118
| | | | | | | In some cases the user running mixlib-shellout needs additional privileges, in other cases the user we're trying to run the process as needs them. Signed-off-by: Bryan McLellan <btm@loftninjas.org>
* Launch process with elevated tokenNimishaS2017-07-193-23/+91
| | | | Signed-off-by: NimishaS <nimisha.sharad@msystechnologies.com>
* fix chefstyle offensesLamont Granquist2017-06-202-24/+26
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* spec cleanup and some light refactoringlcg/cleanup2Lamont Granquist2017-02-012-53/+36
| | | | | | | | | | | | | | | - mostly cleans up the windows specs so its easier to read the API out of them and removes some of the very brittle internal testing - refactors the 'which' logic a bit. trying to converge towards the chef/chef version and eventually extracting common code so that do not have to maintain 10+ slightly different copies everywhere. - adds the Mixlib::ShellOut::EmptyWindowsCommand exception because letting CreateProcessW throw a generic SystemCallError is pretty much useless to everyone. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* remove largely useless Utils classLamont Granquist2017-01-271-120/+126
| | | | | | | | its just a collection of private methods, they should be private methods retain a method in the class that was being used externally. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Resolve most chefstyle warningsTim Smith2016-10-073-101/+101
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Merge pull request #116 from docwhat/commonexceptionsThom May2016-10-071-3/+4
|\ | | | | Modified all exceptions to have a common parent
| * Modified all exceptions to have a common parentChristian HoĢˆltje2016-01-111-3/+4
| | | | | | | | Closes #98
* | bumping 2.2.7 for relase2.2.7Lamont Granquist2016-08-221-1/+1
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | add logging message if attempting to kill critical processsmurawski/agressive_terminationSteven Murawski2016-06-061-1/+9
| |
* | Keep `kill_process_tree` from killing protected system processes.Steven Murawski2016-05-241-13/+25
| |
* | opscode -> ChefTim Smith2016-02-173-11/+11
| |
* | release 2.2.6Matt Wrock2016-02-031-1/+1
| |
* | change CreateProcessAsUserW to use a int instead of boolMatt Wrock2016-02-021-1/+1
|/
* release 2.2.52.2.5Thom May2015-12-021-1/+1
|
* bump to 2.2.4Thom May2015-12-021-1/+1
|
* bumping to 2.2.32.2.3domain_userMatt Wrock2015-10-221-1/+1
|
* add a comment for the recursive functionMatt Wrock2015-10-221-0/+4
|
* improve the test to more accurately detrermine if child process is killedMatt Wrock2015-10-201-3/+7
|
* removing specialized logger and cleaning loggingMatt Wrock2015-10-201-12/+14
|
* merging latest masterMatt Wrock2015-10-192-251/+244
|\
| * Bump to 2.2.22.2.2Kartik Null Cating-Subramanian2015-10-191-1/+1
| |
| * Clean up the monkey-patchingksubrama/win32-processKartik Null Cating-Subramanian2015-10-151-246/+239
| |
| * Pull in upstream changes into monkey-patchKartik Null Cating-Subramanian2015-10-151-14/+14
| |
* | pinning this btranch to v2.0.1Matt Wrock2015-09-291-1/+1
| |
* | log to our own fileMatt Wrock2015-09-281-7/+24
| |
* | add logging for each child process killedMatt Wrock2015-09-241-0/+1
| |
View Lorry Controller Status