| Desired Behavior: Commandline arrays should be treated the same whether the execute method is passed the ':sensitive' option or not. Actual Behavior: In the case statement on lines 165-172 of execution.rb, if the option hash contains the ':sensitive' option, the casting of all array elements to string that occurs on line 168 is not done. The means that later in the execute_windows method when part.include?(' ') is called, there will be an error if one of the array elements is not a string. Examples:
This issue was encountered during the course of the PR to chocolatey to redact sensitive commandline calls.
The type had to be modified to munge all values to the priority property to string, because it is not uncommon to supply integers to that property, which after supplying the ':sensitive' option to the execute method in the provider suddenly started throwing an error. This error seems likely to only be possible on Windows since the `include?` call seems to part of an effort at quote normalization that doesn't seem to be necessary in the excute_posix method. |
