summaryrefslogtreecommitdiff
path: root/lib/chef/mixin
Commit message (Collapse)AuthorAgeFilesLines
* Fixing review commentsnimisha2017-02-021-90/+0
| | | | Signed-off-by: nimisha <nimisha.sharad@msystechnologies.com>
* Fixed bugs, review comments and specsnimisha2017-02-021-6/+15
| | | | Signed-off-by: nimisha <nimisha.sharad@msystechnologies.com>
* Alternate user execute resource code review feedackadamedx2017-02-021-5/+5
|
* Rubocop violations in execute resource alternate user implementationadamedx2017-02-021-1/+1
|
* Windows user identity user name and credntial validationAdam Edwards2017-02-021-0/+81
|
* Merge pull request #4894 from chef/lcg/dnf-providerLamont Granquist2017-01-111-9/+5
|\ | | | | DNF Provider PR #2
| * squashing dnf workLamont Granquist2016-12-131-9/+5
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | cleanup of group providerlcg/group-provider-cleanupLamont Granquist2016-12-211-12/+58
| | | | | | | | | | | | | | | | - consistent use of shell_out_compact! - remove more uses of run_command - some other code cleanup Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | remove run_command from groupaddLamont Granquist2016-12-201-0/+9
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | formally deprecate run_commandLamont Granquist2016-12-201-0/+1
|/ | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* fix Style/SelfAssignmentLamont Granquist2016-12-062-2/+2
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Merge pull request #5547 from chef/lcg/unified-integerLamont Granquist2016-11-161-19/+19
|\ | | | | Core: fix Lint/UnifiedInteger cop
| * simplify some overly complicated powershell codelcg/unified-integerLamont Granquist2016-11-151-19/+19
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * fix Lint/UnifiedInteger copLamont Granquist2016-11-151-1/+1
| | | | | | | | | | | | believe this becomes necessary for ruby 2.4/3.0 Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | Structure deprecations with additional metadatatm/deprecation_with_urlThom May2016-11-162-11/+5
|/ | | | | | | | This adds URLs to each class of deprecation, and correctly prints and formats them for maximum user efficiency. We also provide the URL to the data collector for Visibility to ingest. Signed-off-by: Thom May <thom@chef.io>
* More Opscode -> Chef updatesTim Smith2016-10-201-1/+1
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Fixed double quotes issue in powershell_outNimishaS2016-10-071-1/+1
| | | | Signed-off-by: NimishaS <nimisha.sharad@msystechnologies.com>
* Revert Handling double quotes in powershell_outMatt Wrock2016-10-051-10/+2
| | | | Signed-off-by: Matt Wrock <matt@mattwrock.com>
* Raising Mixlib::ShellOut::ShellCommandFailed exceptionNimishaS2016-10-031-2/+2
| | | | Signed-off-by: NimishaS <nimisha.sharad@msystechnologies.com>
* Handling double quotes. Added specsNimishaS2016-09-301-2/+10
| | | | Signed-off-by: NimishaS <nimisha.sharad@msystechnologies.com>
* Fixed undefined short_cksum method issue and checksum in uppercase issue for ↵Aliasgar162016-09-201-0/+5
| | | | | | | | | | windows_package resource. (#5332) * Fixed undefined short_cksum method issue and checksum in uppercase issue for windows_package resource. * Added RSpecs for the changes done here. * Fixed RSpecs issues.
* Allow flagging a resource property as sensitiveadamleff/sensitive-propertiesAdam Leff2016-08-251-0/+3
| | | | | | | | | | | | Some properties in custom resources may include sensitive data, such as a password for a database server. When the Resource's state is built for use by Data Collector or similar auditing tool, `Chef::Resource#state_for_resource_reporter` builds a hash of all state properties for that resource and their values. This leads to sensitive data being transmitted and potentially stored in the clear. This change enhances properties with the ability to set an individual property as sensitive and then have the value of that property suppressed when exporting the Resource's state.
* clean up everythingLamont Granquist2016-08-231-0/+30
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Stop using obsolete URI.escape; use Addressable::URI instead.John Keiser2016-08-221-1/+2
| | | | See http://stackoverflow.com/questions/2824126/whats-the-difference-between-uri-escape-and-cgi-escape#answer-13059657 for details on why.
* fix Style/BlockDelimiters, Style/MultilineBlockLayout and 0.42.0 engine upgradeLamont Granquist2016-08-173-5/+5
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* fixing Style/MultilineTernaryOperator copLamont Granquist2016-08-161-4/+3
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* make deprecation warnings actual deprecation warningsLamont Granquist2016-04-251-1/+1
|
* remove duping the args hashlcg/shell_out_syntaxLamont Granquist2016-04-251-1/+0
| | | | | we only ever did this in order to mutate the options and with the **options syntax we don't need to do this anymore.
* tighter code from @coderangerLamont Granquist2016-04-251-10/+5
|
* modernize shell_out method syntaxLamont Granquist2016-04-251-16/+11
| | | | **options has worked ever since we've deprecated ruby 1.9.x
* remove global_resource_collectionLamont Granquist2016-04-111-4/+0
|
* readability tweakLamont Granquist2016-04-111-2/+4
|
* fix logic in updating new_resourceLamont Granquist2016-04-111-3/+4
|
* add notifying_block and subcontext_block to chefLamont Granquist2016-04-111-0/+54
|
* Obvious fix: Update lamont@getchef.com email to lamont@chef.ioadamedx/lamont-emailAdam Edwards2016-04-041-1/+1
|
* Fix proxified socketSalim Alam2016-03-211-2/+6
|
* lazy module inclusion into DSL moduleslcg/use-and-lazy-module-includeLamont Granquist2016-03-151-0/+77
| | | | | | Chef::DSL::Recipe::FullDSL.send(:include, MyModule) will now patch all its descendants that it has been included into (works the way actual inheritance works now).
* Merge pull request #4706 from chef/lcg/chefstyle-andorLamont Granquist2016-03-153-3/+3
|\ | | | | chefstyle Style/AndOr fixes
| * chefstyle Style/AndOr fixesLamont Granquist2016-03-143-3/+3
| | | | | | | | | | this is part of our informal style guide, lets make it formal since clearly its not getting followed very well.
* | Issue 4334: Because set(property, nil) gets turned into get(property), add a ↵cd/fix-spurious-warningsChris Doherty2016-03-111-2/+2
|/ | | | 'nil_set' parameter to get() to skip a spurious warning when that happens.
* Fix property coercion in both places, because they were different and Noah Kantrowitz2016-03-091-1/+1
| | | one in Chef::Mixin::Properties would result in duplicated values.
* Merge pull request #4617 from chef/lcg/chefstyle-perfLamont Granquist2016-03-044-5/+8
|\ | | | | Autofixing new Perf cops in 0.37.2
| * add some @yield docsLamont Granquist2016-03-042-0/+2
| | | | | | | | added some FIXMEs because trying to sort out types for random undocumented code was making me completely aggro
| * remove unused block argumentslcg/chefstyle-perfLamont Granquist2016-02-232-2/+2
| |
| * Autofixing new Perf cops in 0.37.2Lamont Granquist2016-02-233-3/+4
| | | | | | | | | | | | | | | | | | | | 6 Performance/Casecmp 18 Performance/Detect 1 Performance/RangeInclude 27 Performance/RedundantBlockCall 6 Performance/RedundantMatch 5 Performance/RedundantMerge 18 Performance/StringReplacement
* | ensure paths maintain utf-8ness in non ascii encodingsumlautMatt Wrock2016-02-241-1/+1
|/
* flip multiline function arguments aroundlcg/rubocop-0.37.2Lamont Granquist2016-02-193-7/+7
| | | | no enforced trailing comma on arguments...
* deal with 0.37.2 renamed copsLamont Granquist2016-02-182-2/+2
| | | | | | | | | | | | | | | 252 Style/TrailingCommaInLiteral 84 Style/TrailingCommaInArguments 15 Style/SpaceAroundKeyword -- 351 Total We already dealt with SpaceAroundKeyword under its old name SpaceBeforeModifierKeyword, it looks like it got stricter about spaces after keywords. TrailingComma also got split, and it looks like the TrailingCommaInArguments behavior is new?
* convert MultilineOperationIndentation style to "indented"Lamont Granquist2016-02-122-2/+2
| | | | this looks nicer.
* pull rubocop 0.37.2 into chefstyleLamont Granquist2016-02-122-6/+6
| | | | | this is from the same ruleset that we had, but the new code catches more conditions.
View Lorry Controller Status