* modify resource
* windows compatibility
* end line
* removing comments to clean things up
Signed-off-by: Aaron Lippold <lippold@gmail.com>
* CHEF-12175: Remove use of `Kernel.open` or `IO.read` or similar sinks with a non-constant value (#7079)
* fix: update io reads to fix codescanning vuln
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* fix: remove kernel#open fallback and update open method to handle local files
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* fix: replace io.readlines to file.readlines
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* chore: fix lint offense, unnecessary space
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* fix: raise exception for standard errors as FetcherFailure
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* chore: update exception message
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
---------
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* fix: Missing regular expression anchor (#7077)
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* Bump version to 6.6.27 by Chef Expeditor
Obvious fix; these changes are the result of automation not creative thinking.
* Fix gitignore file in docs dir (#7082)
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
* Bump version to 6.6.28 by Chef Expeditor
Obvious fix; these changes are the result of automation not creative thinking.
* adding docs now that the gitignore is fixed.
Signed-off-by: Aaron Lippold <lippold@gmail.com>
* add docs for sshd_active_config resource
* fix docs
* unit tests
* rubocop on ssh_config.rb, sshd_con_test.rb, and sshd_active_config.md
* rename active path in method
* doc fix
* Linting Corrections for Active SSHD Config (#1)
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
* Update sshd_active_config.md
updated version
* doc version update
---------
Signed-off-by: Aaron Lippold <lippold@gmail.com>
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
Co-authored-by: Daniel Medina <dmedina@mitre.org>
Co-authored-by: Sonu Saha <98935583+ahasunos@users.noreply.github.com>
Co-authored-by: Chef Expeditor <chef-ci@chef.io>
Co-authored-by: Ian Maddaus <IanMadd@users.noreply.github.com>
Co-authored-by: Clinton Wolfe <clintoncwolfe@gmail.com>
* Remove unused telemetry v1 code
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
* Sketch out basics of telemetry, with start/stop of invocation telemetry
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
* Data structure for run telemetry - job capture
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
* Add per-control and per-run feature detection
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
* CHEF-4017 Telemetry job api updations (#6965)
* Added initial changes to jobs api
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Feature flag changes for telemetry
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* move base, debug and null to its own file structure
Signed-off-by: Sathish <sbabu@progress.com>
* make HTTP client post requests
Signed-off-by: Sathish <sbabu@progress.com>
* remove old logic
Signed-off-by: Sathish <sbabu@progress.com>
* make backend class as `HTTP`
Signed-off-by: Sathish <sbabu@progress.com>
* CHEF-7258 Fetch and use licensing information for telemetry (#6964)
* Added method to fetch license ids for inspec
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Added free license check for performing telemetry api call
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* move base, debug and null to its own file structure
Signed-off-by: Sathish <sbabu@progress.com>
* make HTTP client post requests
Signed-off-by: Sathish <sbabu@progress.com>
* remove old logic
Signed-off-by: Sathish <sbabu@progress.com>
* make backend class as `HTTP`
Signed-off-by: Sathish <sbabu@progress.com>
---------
Signed-off-by: Nik08 <nikita.mathur@progress.com>
Signed-off-by: Sathish <sbabu@progress.com>
Co-authored-by: Sathish <sbabu@progress.com>
* Updated control tags and desc value to be used in jobs api
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Added checks for automate run context and free license check
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* capture target mode and id
Signed-off-by: Sathish <sbabu@progress.com>
* profile doesn't need ID
Signed-off-by: Sathish <sbabu@progress.com>
* use run context to set environment data
Signed-off-by: Sathish <sbabu@progress.com>
* refactor `create_wrapper` to be localized
Signed-off-by: Sathish <sbabu@progress.com>
* change all timestamps to be UTC
Signed-off-by: Sathish <sbabu@progress.com>
* Null checks for response and corrected job api endpoint
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Fixed tag values to be sent as string in api call
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* make version as float
Signed-off-by: Sathish <sbabu@progress.com>
* add platform name
Signed-off-by: Sathish <sbabu@progress.com>
* Added control result data in jobs api payload
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Debug logs added for telemetry call
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Removed unwanted telemetry debug class
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Payload fix to pass features data only on per control basis
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Added class function to list all invoked features by feature sub system
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Using feature system to get all invoked features list to be used in jobs api
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Unit tests cases updated and fixed
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* License type check downcased
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Lint fix
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* CHEF-7265 Telemetry opt-in for CINC users (#6966)
* Enabled telemtry opt-in
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Removed old comments
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Unit test case added to validate the disabling telemetry behaviour for inspec user
Signed-off-by: Nik08 <nikita.mathur@progress.com>
---------
Signed-off-by: Nik08 <nikita.mathur@progress.com>
---------
Signed-off-by: Nik08 <nikita.mathur@progress.com>
Signed-off-by: Sathish <sbabu@progress.com>
Co-authored-by: Sathish <sbabu@progress.com>
* Product team review changes - only disable telemetry for commercial license users
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Connection failure handling for telemetry http call
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Testing fix - Remove usage of deleted library
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Telemetry test case fix - Issue caused because unit test are run without feature flag env set
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Fixed and replaced tightly coupled semver versioning regex matching test for telemetry data
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Telemery test fix to use license key from env or a dummy value if not set in env
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Added error logs in case the http call is not successful for telemetry
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Error handling for telemetry start and run calls
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Telemetry opt-in changes (#7055)
* Removed usage of feature system to enable telemetry - making it opt-in by default
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Telemetry disable check fix when no option is passed in args
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Fix in test to use license specs defined for testing
Signed-off-by: Nik08 <nikita.mathur@progress.com>
---------
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* (Restoring) CHEF-10392 load default telemetry url conditionally (#7059)
* load default telemetry url conditionally
Signed-off-by: Sathish <sbabu@progress.com>
* remove version base path
version base path is defined in jobs path already
Signed-off-by: Sathish <sbabu@progress.com>
* use `CHEF_` prefix for the ENV
Signed-off-by: Sathish <sbabu@progress.com>
---------
Signed-off-by: Sathish <sbabu@progress.com>
Co-authored-by: Sathish <sbabu@progress.com>
* Typo fix in features list
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Stub added for CI license key
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* License usage telemetry correction - not track control results (#7060)
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Changes to disable telemetry for other InSpec distros (#7065)
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Lint issue fix
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Removing disable telemetry test - breaks on CI because of commercial license usage
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* CHEF-13228 Chef licensing telemetry documentation (#7056)
* WIP chef telemetry env variable usage updated
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* WIP intro added for chef telemetry - requires edit
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Correction in opt in behaviour of telemetry
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Doc update after default opt in changes
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Doc edit from product
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Doc edit
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Edits
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
---------
Signed-off-by: Nik08 <nikita.mathur@progress.com>
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
Co-authored-by: Ian Maddaus <ian.maddaus@progress.com>
* Updated version pinning of chef licensing to version 1 for chef telemetry
Signed-off-by: Nik08 <nikita.mathur@progress.com>
---------
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
Signed-off-by: Nik08 <nikita.mathur@progress.com>
Signed-off-by: Sathish <sbabu@progress.com>
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
Co-authored-by: Nikita Mathur <Nik08@users.noreply.github.com>
Co-authored-by: Sathish <sbabu@progress.com>
Co-authored-by: Nik08 <nikita.mathur@progress.com>
Co-authored-by: Ian Maddaus <ian.maddaus@progress.com>
* Fixes the sybase_session resource was not working with c shell
Signed-off-by: Vasu1105 <vjagdale@progress.com>
* Adds TODO
Signed-off-by: Vasu1105 <vjagdale@progress.com>
---------
Signed-off-by: Vasu1105 <vjagdale@progress.com>
* fix: Incomplete string escaping or encoding
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* fix: Incomplete string escaping or encoding
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* fix: Incomplete string escaping or encoding
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* chore: fix lint offense for single quotes
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
---------
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* fix: update io reads to fix codescanning vuln
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* fix: remove kernel#open fallback and update open method to handle local files
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* fix: replace io.readlines to file.readlines
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* chore: fix lint offense, unnecessary space
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* fix: raise exception for standard errors as FetcherFailure
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* chore: update exception message
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
---------
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* Add configurable InSpec backend for Chef Infra
Signed-off-by: Thomas Heinen <thomasheinen@gmail.com>
* Add changing backend for Chef Infra
Signed-off-by: Thomas Heinen <thomasheinen@gmail.com>
* Add test
* Fix typo on test
Signed-off-by: Thomas Heinen <thomasheinen@gmail.com>
* Fix test
Signed-off-by: Thomas Heinen <thomasheinen@gmail.com>
---------
Signed-off-by: Thomas Heinen <thomasheinen@gmail.com>
* fix: remove --symref options from git ls-remote
it is causing issue on ubuntu16 and rhel6
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* spec: update test to match commit-sha.tar.gz instead of master.tar.gz
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* spec: skip test on windows, works locally on windows
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
---------
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
parser 3.3.1.0 introduced
https://github.com/whitequark/parser/pull/1000, which causes this
failure:
```
% bundle exec ruby -Itest test/unit/profiles/profile_test.rb
inspec/lib/inspec/utils/profile_ast_helpers.rb:7:in `<class:CollectorBase>': uninitialized constant Parser::AST::Processor::Mixin (NameError)
include Parser::AST::Processor::Mixin
^^^^^^^
```
Fix this by inherting from `Parser::AST::Processor` instead.
Closes#7029
Signed-off-by: Stan Hu <stanhu@gmail.com>
* fix: use git gem to perform git remote show
this fixes the issue to perform git remote show when dir is not a git dir
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* test: update test with default_ref stubbing
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* chore: remove stubs which are not required
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* chore: remove unused method
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* test: fix broken test due to changes in url fetcher
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* chore: improve error message to be user-friendly
this makes the error message as earlier one
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* chore: change error message:
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* revert: remove git gem dependency and use shellout
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* fix: update url fetcher to use ls-remote to make it work in non-git dirs
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
---------
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* Added class function to list all invoked features by feature sub system
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Test cases to verify invoked features list function
Signed-off-by: Nik08 <nikita.mathur@progress.com>
---------
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Adds the check for custom plugins to avoid showing "Unrecognised Feature " warnings for them.
Signed-off-by: Vasu1105 <vjagdale@progress.com>
* Test to verify feature validation should not show warning for custom plugins
Signed-off-by: Vasu1105 <vjagdale@progress.com>
* Fix the test failures
Signed-off-by: Vasu1105 <vjagdale@progress.com>
---------
Signed-off-by: Vasu1105 <vjagdale@progress.com>
* Allow the negate formating for cmp
The 'cmp' matcher when used with a negation as 'should_not' does
has a confusion formating when control fails:
expected: 3
got: 3
This, because the 'format_actual' always pass 'false' to the
'format_expectation'. Thus the 'format_expectation' never reach the
'negate_str'.
Signed-off-by: Pg <pg.developper.fr@gmail.com>
* Linting
Signed-off-by: Pg <pg.developper.fr@gmail.com>
Co-authored-by: Vasundhara Jagdale <vjagdale@progress.com>
---------
Signed-off-by: Pg <pg.developper.fr@gmail.com>
Co-authored-by: Vasundhara Jagdale <vjagdale@progress.com>
* test: add unit test for inspec sign #profile_verify
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* enhance: add support for custom config dir to be used during testing
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* test: add unit test for inspec sign #keygen and #profile_sign
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* test: remove test for profile sign
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* test: add test for methods in inspec sign
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
---------
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* chore: change log-level for url fetcher transformation
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* test: remove warning logs from url fetcher test
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
---------
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* Fix the undefined method file_read error which is causing virtualizaiton resource to set nil for virtualization platform check
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Adds unit test for virtualization resource detect podman container usind detect_container method.
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
---------
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Initial commit to extend backward compatibilty support of waiver with chef client
Signed-off-by: Nik08 <nikita.mathur@progress.com>
* Changes in conditional checks using if instead of unless
Signed-off-by: Nik08 <nikita.mathur@progress.com>
---------
Signed-off-by: Nik08 <nikita.mathur@progress.com>
