mirror of
https://github.com/inspec/inspec
synced 2024-11-22 20:53:11 +00:00
2613 commits
| Author | SHA1 | Message | Date | |
|---|---|---|---|---|
Gokulakrishnan KS
|
14f546391f
|
Added fatal message check for postgres_session:query method (#7154)
* Added fatal message check for postgres_session:query method Signed-off-by: Gokulakrishnan KS <Gokulakrishnan.KS@progress.com> * Added empty new line for the new fixture Signed-off-by: Gokulakrishnan KS <Gokulakrishnan.KS@progress.com> --------- Signed-off-by: Gokulakrishnan KS <Gokulakrishnan.KS@progress.com> |
||
Vasundhara Jagdale
|
7f1174a753
|
Attempt to fix the random failure of depedency_loader unit test (#7101)
* Attempt to fix the dependency_loader random unit test failure in verify pipeline
Signed-off-by: Vasu1105 <vjagdale@progress.com>
* Revert "Attempt to fix the dependency_loader random unit test failure in verify pipeline"
This reverts commit
|
||
Aaron Lippold
|
0ab35681aa
|
Update ssh_config resource to support returning the current sshd_config file in use. (#7070)
* 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> |
||
Clinton Wolfe
|
29242deb7c
|
Usage Telemetry v3 (#6012)
* 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> |
||
Thomas Heinen
|
ea7e386028
|
Configurable backend for Chef Infra in Target Mode (#7058)
* 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> |
||
|
Vasundhara Jagdale
|
26e6cbfb9c
|
Remove Ruby 3.0 support (#7057)
* Remove Ruby 3.0 support Signed-off-by: Vasu1105 <vjagdale@progress.com> * Updates sonarqube coverage file Signed-off-by: Vasu1105 <vjagdale@progress.com> * Fix failing test after remmoval of support for ruby 3.0 Signed-off-by: Vasu1105 <vjagdale@progress.com> --------- Signed-off-by: Vasu1105 <vjagdale@progress.com> |
||
Sonu Saha
|
c8f020cb81
|
Modify url fetcher to remove --symref flag in git ls-remote command (#7043)
* 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> |
||
|
Sonu Saha
|
65bd83f6ed
|
CHEF-6000: Fix URL fetcher failure to work when current directory is not a Git repository (#7023)
* 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> |
||
Nikita Mathur
|
61a4ee3fc2
|
List all invoked features by feature sub system (#6979)
* 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> |
||
|
Sonu Saha
|
698df457e2
|
test: update test with updated failure message for gcp auth (#7010)
Signed-off-by: Sonu Saha <sonu.saha@progress.com> |
||
|
Vasundhara Jagdale
|
91210f8afe
|
CHEF-9848 - Exclude Warning "Unrecognized feature name" for Custom InSpec plugins. (#6988)
* 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> |
||
|
Vasundhara Jagdale
|
c4e5386656
|
Adds unit test for progress bar streaming reporter (#6970)
Signed-off-by: Vasu1105 <vjagdle@progress.com> Co-authored-by: Vasu1105 <vjagdle@progress.com> |
||
|
Sonu Saha
|
16a74f39ae
|
CHEF-3162: Implement unit test for inspec-sign plugin (#6959)
* 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> |
||
|
Sonu Saha
|
d4208cb63c
|
CHEF-4571: Silence warning about transforming into git fetcher (#6946)
* 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> |
||
|
Vasundhara Jagdale
|
ebd3eeca55
|
CHEF-6412: Fix for virutalization resource undefined method file_read (#6947)
* 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> |
||
|
Vasundhara Jagdale
|
dc7631b8ac
|
Skip audit log test on Windows (#6950)
* Skip audit log test on Windows Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Fix the text in the audit log test Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> --------- Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> |
||
|
Nikita Mathur
|
7ac8ccf7c1
|
Unit tests added for enhanced outcomes (#6924)
Signed-off-by: Nik08 <nikita.mathur@progress.com> |
||
|
Nikita Mathur
|
75fe6a0e27
|
Unit tests added for iaf_file (#6933)
Signed-off-by: Nik08 <nikita.mathur@progress.com> |
||
|
Vasundhara Jagdale
|
e48e0c7786
|
Fix failing ssh_key test on Windows (#6948)
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> |
||
|
Vasundhara Jagdale
|
b946f5454d
|
CHEF-3962: Adds ability to verify ssh keys (#6656)
* ssh_key resource file Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Initial commit for unit test for ssh_key resource Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Fix linting and added resource in resources file Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * extend the ssh key resource to use file resource properties Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Updates the ssh_key resource to get length and type of key. Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Updates unit test for ssh_key resource. Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * ADDS Docs for ssh_key resource Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Fix Review: Empty file handling Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Fixed review comments and few code refactoring for ssh_key resource Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Doc edits Signed-off-by: Ian Maddaus <ian.maddaus@progress.com> * Updates test Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> --------- Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> Signed-off-by: Ian Maddaus <ian.maddaus@progress.com> Co-authored-by: Ian Maddaus <ian.maddaus@progress.com> |
||
|
Nikita Mathur
|
18847ac69d
|
Fix CI test for windows (#6913)
* Skip functional tests to match exit code for mandatory profile signing error in windows Signed-off-by: Nik08 <nikita.mathur@progress.com> * Addded comment to clarify skipping exit code matching test in windows Signed-off-by: Nik08 <nikita.mathur@progress.com> --------- Signed-off-by: Nik08 <nikita.mathur@progress.com> |
||
|
Sonu Saha
|
b79ad46e5a
|
Fix examples for file resource provided in docs (#6899)
* Update docs and introduce test for file content Signed-off-by: Sonu Saha <sonu.saha@progress.com> * TEST: Fix broken test due to changes in fixture Signed-off-by: Sonu Saha <sonu.saha@progress.com> --------- Signed-off-by: Sonu Saha <sonu.saha@progress.com> |
||
|
Vasundhara Jagdale
|
20d0a9dc0d
|
CHEF-6440: Integrates audit logging feature (#6809)
* Adds cli options to enable audit log and configure the audit log Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Updated code to validate the audit log options. Audit log options are only valid for inspec exec and inspec shell command as those commands use the backend to execute commands and for file operations Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Updates cli options documentation for audit log options Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Disable audit log in test environment unless and until explicitly --enable-audit-log option is provided in test Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Rename enable-audit-log cli option to disable-audit-log Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * CHEF-8210 Enables feature preview flag for audit logging Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Revert "Disable audit log in test environment unless and until explicitly --enable-audit-log option is provided in test" This reverts commit 50a42f0b44cf2fde6d927a00e444370cdd288a5d. * Adds functional test for audit logging feature Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Updates cli doc for audit loggin feature flag Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Debug why audit log functional test fails in ci environment Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Revert "Rename enable-audit-log cli option to disable-audit-log" This reverts commit |
||
|
Nikita Mathur
|
fd4e6d97a6
|
CHEF-6439 Mandatory Profile Signing (Preview) (#6705)
* Updated exec option to allow unsigned profiles run Signed-off-by: Nik08 <nikita.mathur@progress.com> * Added method to verify signed profile and to check for signed profile Signed-off-by: Nik08 <nikita.mathur@progress.com> * Invoked logic on each run to verify profiles if signed else raise sig req error Signed-off-by: Nik08 <nikita.mathur@progress.com> * Tests cases added to validate behaviour of inspec exec with signed and unsigned profiles with --chef-allow-unsigned flag Signed-off-by: Nik08 <nikita.mathur@progress.com> * Refactored and moved delete_signing_keys to common helper library for tests Signed-off-by: Nik08 <nikita.mathur@progress.com> * Updated code comments for more information and clarity on security update of signed profiles inspec exec Signed-off-by: Nik08 <nikita.mathur@progress.com> * Test cases to validate inspec run with combination of signed and unsigned profiles Signed-off-by: Nik08 <nikita.mathur@progress.com> * Documented usage of flag --chef-allow-unsigned Signed-off-by: Nik08 <nikita.mathur@progress.com> * Renamed the flag to run unsigned profiles to --allow-unsigned Signed-off-by: Nik08 <nikita.mathur@progress.com> * Refactored logic on profile level for profile signing verification Signed-off-by: Nik08 <nikita.mathur@progress.com> * Renaming the argument variable - from runner_call to silent Signed-off-by: Nik08 <nikita.mathur@progress.com> * Added profile mandate check for other inspec commands running profile evaluation Signed-off-by: Nik08 <nikita.mathur@progress.com> * Updated error message for profile sign requirement Signed-off-by: Nik08 <nikita.mathur@progress.com> * Updated test helper to fix inspec json test Signed-off-by: Nik08 <nikita.mathur@progress.com> * Fixed inspec json ability to use cli options successfully Signed-off-by: Nik08 <nikita.mathur@progress.com> * Documentation added for signed profiles mandatory usage with CLI commands Signed-off-by: Nik08 <nikita.mathur@progress.com> * Flow changes of raising exception when unsigned instead of direct exit Signed-off-by: Nik08 <nikita.mathur@progress.com> * Renamed unsigned profile flags Signed-off-by: Nik08 <nikita.mathur@progress.com> * Extracted out allow unsigned condition to config and modified comment info Signed-off-by: Nik08 <nikita.mathur@progress.com> * Doc update on consent of using signed and unsigned profiles Signed-off-by: Nik08 <nikita.mathur@progress.com> * Fix in signing mandatin check and added additional check on runner for better error UI for exec command Signed-off-by: Nik08 <nikita.mathur@progress.com> * Removed repeated allow-unsigned-profile defination from exec_options Signed-off-by: Nik08 <nikita.mathur@progress.com> * Test fixes Signed-off-by: Nik08 <nikita.mathur@progress.com> * Enabled feature preview flag for mandatory signing Signed-off-by: Nik08 <nikita.mathur@progress.com> * Test fixes after feature flag usage for mandatory signing Signed-off-by: Nik08 <nikita.mathur@progress.com> * Doc changes using feature preview flag for mandatory signing feature Signed-off-by: Nik08 <nikita.mathur@progress.com> * Inspec exec tests fixes for ENV values and parallel test fix using default option --allow-unsigned-profile false Signed-off-by: Nik08 <nikita.mathur@progress.com> * Kitchen fix while using signed profiles with inspec Signed-off-by: Nik08 <nikita.mathur@progress.com> * Unit test fix for profile resource exception Signed-off-by: Nik08 <nikita.mathur@progress.com> * Virtual profile detection improved Signed-off-by: Nik08 <nikita.mathur@progress.com> * Move mandatory profile sigining info to sigining page Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Renamed flag from --allow-unsigned-profile to --allow-unsigned-profiles Signed-off-by: Nik08 <nikita.mathur@progress.com> * Typo fix in signing doc Signed-off-by: Nik08 <nikita.mathur@progress.com> * Trim note in cli.md about mandatory profile signing Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Docs changes Signed-off-by: Ian Maddaus <ian.maddaus@progress.com> * Correct docs regarding exit code 5 Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> --------- Signed-off-by: Nik08 <nikita.mathur@progress.com> Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> Signed-off-by: Ian Maddaus <ian.maddaus@progress.com> Co-authored-by: Clinton Wolfe <clintoncwolfe@gmail.com> Co-authored-by: Ian Maddaus <ian.maddaus@progress.com> |
||
|
Vasundhara Jagdale
|
ed033b9ac2
|
CHEF-7694 (CHEF-8271) : Fix security_policy resource returns array for non comma separated values (#6838)
* Adds unit test for more security policies Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Fix for secuity_policy resource failing if policy includes single value in string format it returns array instead of string Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Update code comments Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> --------- Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> |
||
|
Sonu Saha
|
b5fcc141d2
|
CHEF-6437: Implement different version of inspec export (#6816)
* Failing test for export - should not evaluate Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Sketch out a info_from_parse method Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Temporary commit to checkpoint experimental work Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Basic control ids extraction Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Modify to capture entire block Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Ability to parse desc, impact and title of a control (#6662) Signed-off-by: Sonu Saha <sonu.saha@progress.com> * Rework per-control metadata collectors to be class-based Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * REFACTOR: make a common base class for collectors Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * memoise `info_from_parse` Signed-off-by: Sathish <sbabu@progress.com> * Add --legacy-export option to inspec export (#6661) * support legacy export option Signed-off-by: Sathish <sbabu@progress.com> * ability to run legacy export option Signed-off-by: Sathish <sbabu@progress.com> --------- Signed-off-by: Sathish <sbabu@progress.com> * Improve ControlIDCollector and other fields of export data (#6686) * Parse tags & refs from the ast nodes Signed-off-by: Sonu Saha <sonu.saha@progress.com> * ENHANCE: Improve Desc collector to collect description Signed-off-by: Sonu Saha <sonu.saha@progress.com> * ENHANCE: Only loop through the child node of begin block Signed-off-by: Sonu Saha <sonu.saha@progress.com> * FIX: Fix bug/todo to handle duplicacy of control ids Signed-off-by: Sonu Saha <sonu.saha@progress.com> * TEST - a profile which fails to properly be exported but is likely to be used by MITRE Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Revert "FIX: Fix bug/todo to handle duplicacy of control ids" This reverts commit |
||
|
Nikita Mathur
|
456fa5c8f5
|
CHEF-7670 Ability to set feature preview flag using ENV (#6833)
* Implemented ability to add feature flags in Inspec feature subsystem Signed-off-by: Nik08 <nikita.mathur@progress.com> * Added usage of env_preview for conditionalising features Signed-off-by: Nik08 <nikita.mathur@progress.com> * Test cases added for feature preview flag ability Signed-off-by: Nik08 <nikita.mathur@progress.com> * Minor chnages - Better commenting & smarter methods Signed-off-by: Nik08 <nikita.mathur@progress.com> --------- Signed-off-by: Nik08 <nikita.mathur@progress.com> |
||
|
Nikita Mathur
|
b360d64a1c
|
Changed params usage in inspec archive to skip evaluation (#6741)
* Using metadata to fetch basic info as name and version required in archive Signed-off-by: Nik08 <nikita.mathur@progress.com> * Typo fixes Signed-off-by: Nik08 <nikita.mathur@progress.com> --------- Signed-off-by: Nik08 <nikita.mathur@progress.com> |
||
Sathish Babu
|
e1d39e581a
|
CHEF-6422 Modify inspec archive to not check or export by default (#6659)
* make profile check optional Signed-off-by: Sathish <sbabu@progress.com> * Clarify error message when making inspec check optional Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Add test fixture profile that emits evaluation markers on stderr Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Make export to inspec.json optional, not enabled by default Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Add test confirming non-evaluative archive Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Fix failing plugin search test, failing since we published a new test gem Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> --------- Signed-off-by: Sathish <sbabu@progress.com> Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> Co-authored-by: Clinton Wolfe <clintoncwolfe@gmail.com> |
||
|
Vasundhara Jagdale
|
95c17d4e7f
|
CHEF-3849: InSpec should exit quickly and clearly if waiver file is malformed/corrupt (#6644)
* Functional test for malformed waiver file Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Raise error for malformed yaml content and exit Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Updates functional test for malformed yaml waiver file and for empty waiver file Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Raises error in case of missing required parameters in waiver file Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Update functional test for missing parameters, extra parameters or column without headers in waiver file Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Fix linting Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Fix warning and error messages Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Updates nil and false check for yaml data and adds additional empty check. Co-authored-by: Sathish Babu <80091550+sathish-progress@users.noreply.github.com> Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Adds more generic message as this yaml reader is now getting used by other functionalities like waiver file Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Fixed test description to reflect correct use case Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Fix validate headers was not validating the required parametes for all the data fields as it was not called inside the loop where we are iterating over the data and fetching the headers. Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Updates the test files for the use case to missing parameters and extra parameters Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Updates code to better handle errors and warnings related to missing required parameters and extra parameters in waivers file in all format i.e (yaml, json and csv). Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Updated functional test to capture the updated error and warning messages for waiver file validation Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Fix linting Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Deleted fixture file which is not required Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Refactor: Renamed method validate_headers to reflect whats it's doing and instead of return data in array it will now return the data in hash Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> --------- Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> Co-authored-by: Sathish Babu <80091550+sathish-progress@users.noreply.github.com> |
||
|
Vasundhara Jagdale
|
8ee7bc946d
|
CHEF-642 Fix for inspec exec fails with git fetcher if current directory does not have .git directory (#6626)
* Functional Test to verify git fetcher able to work in a directory where git is not initialized or does not contain the .git directory Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Fix: If git branch, tag or reference information is not given it should clone the repo and checkout the default HEAD Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Remove the logic to find default branch in case of branch, tag or ref is not provided as it's not required logically as we will be cloning the and checkout the default HEAD Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * This is a temporary change to make verify test working on this PR. Revert once the work is done for the git fetcher fix Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Unit Test: Fix unit test Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Verifies the git url before creating cache_key, if given relative path does not exist make sure it deletes the cache directory created to copy the cloned repo. It raises error if any git command fails Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Reverted verify git repo url call from cache_key as cache_key and shifted that to resolve_ref which is already raising the error in case of git ls remote command fails Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Unit test to cover the changes done for resolve_ref when branch, tag or ref is not given and unit test for invalid git repo Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> --------- Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> |
||
|
Sathish Babu
|
5d0ae06a01
|
Missing changes from main (#6564)
* Sync up the default branch as main (#6239) Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com> Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com> * Bump version to 5.20.2 by Chef Expeditor Obvious fix; these changes are the result of automation not creative thinking. * resolves inspec/inspec#6042 (#6043) Signed-off-by: Marijus Ravickas <marijus.ravickas@gmail.com> Signed-off-by: Marijus Ravickas <marijus.ravickas@gmail.com> * Enhance `lxc` resource to test properties (#6243) * ENHANCE: Extend lxc resource to test the properties Signed-off-by: Sonu Saha <sonu.saha@progress.com> * TEST: Extend test for introduced properties Signed-off-by: Sonu Saha <sonu.saha@progress.com> * DOCS: Extend test for introduced properties Signed-off-by: Sonu Saha <sonu.saha@progress.com> * FIX: Correct exception class to handle bad yaml Signed-off-by: Sonu Saha <sonu.saha@progress.com> * Doc Review Signed-off-by: Deepa Kumaraswamy <dkumaras@progress.com> * FIX: Handle specific exception to handle bad yaml Signed-off-by: Sonu Saha <sonu.saha@progress.com> Signed-off-by: Sonu Saha <sonu.saha@progress.com> Signed-off-by: Deepa Kumaraswamy <dkumaras@progress.com> Co-authored-by: Deepa Kumaraswamy <dkumaras@progress.com> * Bump version to 5.20.3 by Chef Expeditor Obvious fix; these changes are the result of automation not creative thinking. * CFINSPEC-462: Fixes inspec sign breaks when there is period or dot in the profile name (#6261) Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Bump version to 5.20.4 by Chef Expeditor Obvious fix; these changes are the result of automation not creative thinking. * CFINSPEC-374 DSL keyword `only_applicable_if` added (#6229) * Not applicable if logic addition Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Changes from not applicable if to only applicable if Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Fix to enable placing only_applicable_if at any position in control and for keeping impact zero intact Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Doc change added for only_applicable_if Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Doc Review Signed-off-by: Deepa Kumaraswamy <dkumaras@progress.com> * Added generic examples that supports cross platform for testing Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Yet another build fix due to changes in test Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Signed-off-by: Deepa Kumaraswamy <dkumaras@progress.com> Co-authored-by: Deepa Kumaraswamy <dkumaras@progress.com> * Bump version to 5.21.0 by Chef Expeditor Obvious fix; these changes are the result of automation not creative thinking. * CFINSPEC-458 Oracle DB session resource compatibility with AIX-C shell (#6257) * Oracle DB session resource compatibility with AIX-Cshell Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Fixes in oracledb session unit test using oracle_query_string Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Bump version to 5.21.1 by Chef Expeditor Obvious fix; these changes are the result of automation not creative thinking. * Replace faraday_middleware with faraday-follow_redirects (#6238) faraday_middleware was already DEPRECATED in 2022-01. Only FaradayMiddleware::FollowRedirects from faraday_middleware has been used, and so we can replace it with faraday-follow_redirects. faraday-follow_redirects 0.3.0 requires faraday ~> 1.0, which is as same as "faraday_middleware 1.0.0 or higher. Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com> Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com> * Bump version to 5.21.2 by Chef Expeditor Obvious fix; these changes are the result of automation not creative thinking. * Update Faraday gem (#6267) Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Bump version to 5.21.3 by Chef Expeditor Obvious fix; these changes are the result of automation not creative thinking. * Bump test-kitchen from 3.3.1 to 3.3.2 in /omnibus (#6227) Bumps [test-kitchen](https://github.com/test-kitchen/test-kitchen) from 3.3.1 to 3.3.2. - [Release notes](https://github.com/test-kitchen/test-kitchen/releases) - [Changelog](https://github.com/test-kitchen/test-kitchen/blob/main/CHANGELOG.md) - [Commits](https://github.com/test-kitchen/test-kitchen/commits) --- updated-dependencies: - dependency-name: test-kitchen dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump version to 5.21.4 by Chef Expeditor Obvious fix; these changes are the result of automation not creative thinking. * Bump omnibus-software from `1d540dc` to `7d0e0fe` in /omnibus (#6274) Bumps [omnibus-software](https://github.com/chef/omnibus-software) from `1d540dc` to `7d0e0fe`. - [Release notes](https://github.com/chef/omnibus-software/releases) - [Commits]( |
||
|
Clinton Wolfe
|
eff13369f1
|
CHEF-4115 Added ability to merge reporter configurations from both CLI and config (#6568) (#6579)
* Added ability to merge cli and config reporter options * Test cases to validate working of reporter configuration using cli and config * Documentation change to add information on reporter configurations usage with both * Added Doc review changes and text fixture for config json * Verify fix * Verify pipeline test fixes for reporter options to be read successfully * Test changes in both cli and config reporter usage scenarios to fix verify pipeline * Review comments to improvise * Renamed testing fixture file for reporter cli config file --------- Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Co-authored-by: Nikita Mathur <Nik08@users.noreply.github.com> |
||
|
Vasundhara Jagdale
|
dfb0172992
|
Adds test for licensing_config (#57)
* Adds test for licensing_config Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> Fix lint Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> Read the value of license server url from env for test Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Spell InSpec corrrectly in test Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Linting Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> --------- Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> Co-authored-by: Clinton Wolfe <clintoncwolfe@gmail.com> |
||
|
Clinton Wolfe
|
a3028b1fc8
|
For #6493 : Add postgres_session support for custom port with a socket connection (#6494) (#40)
* Add support for custom port with a socket connection * Add tests for the postgres_session with custom port --------- Signed-off-by: Pg <pg.developper.fr@gmail.com> Co-authored-by: Pg <pg.developper.fr@gmail.com> |
||
|
Sathish Babu
|
4ec735d09d
|
CHEF 83 Revert attestations changes (#47)
* revert attestation related files Signed-off-by: Sathish <sbabu@progress.com> * revert attestation changes to existing files Signed-off-by: Sathish <sbabu@progress.com> * update signature Signed-off-by: Sathish <sbabu@progress.com> --------- Signed-off-by: Sathish <sbabu@progress.com> |
||
|
Sonu Saha
|
b4648e0193
|
CHEF-1458 Multiple values changes in SimpleConfig library (#6489) (#28)
* Simple config changes to return array of values with multiple values opt * New option added to split multiple values with a delimiter regex --------- Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Co-authored-by: Nikita Mathur <Nik08@users.noreply.github.com> |
||
|
Clinton Wolfe
|
4fce6845e5
|
Add nftables resources (#6499) (#44)
* Fix systemd path for Leap image
* Use vhef client version 17 as doocker cookbook do not support >= 18
* Add nftables resource
* Add nftables tests
* Add fixtures for nftables tests
* enable nftables only when attr is true - then disable iptables
* By default test iptables, not nftables
* Fix tests and lint errors
* Increase unit test coverage for nftables
* Do not use -nn nft option as behaviour changes based on nft version
* Base nft params identification on its version, not os version
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
* Make test more human friendly by reversing unless/if logic
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
* Update mocked cmds with nft params
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
* Fix quoting issue with rubocop
* Fix uninitiallized class vars
* Fix unit test by adding nft version mocking
* Clean nftables doc
---------
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
Co-authored-by: jjacque <jeremy.jacque@algolia.com>
|
||
|
Clinton Wolfe
|
dd9dccd84e
|
Update host resource to resolve all ipaddresses (#6481) (#39)
Signed-off-by: Alan Foster <alan@alanfoster.me> Co-authored-by: Alan Foster <alan@alanfoster.me> |
||
|
Clinton Wolfe
|
1ab146d028
|
fix: ensure Invoke-WebRequest headers can be configured (#6484) (#41)
* fix: ensure Invoke-WebRequest headers can be configured * fix: use array for custom headers * test: add unit test for Windows simple HTTP request with headers --------- Signed-off-by: Adrian Mlodzianowski <amlodzianowski@logicworks.net> Co-authored-by: amlodzianowski <adrianmlodzianowski@gmail.com> Co-authored-by: Adrian Mlodzianowski <amlodzianowski@logicworks.net> |
||
|
Clinton Wolfe
|
cfde3f5bd8
|
CHEF-2438 Add train-kubernetes to inspec gemspec (#6512) (#43)
* Add train-kubernetes to inspec gemspec, fix spacing * Fix failing test to detect system plugins --------- Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> Co-authored-by: Vasu1105 <vasundhara.jagdale@chef.io> |
||
|
Vasundhara Jagdale
|
699d73993c
|
Licensing - Integrates Software Entitlement (#13)
* CFINSPEC-24:Integrate License Software Entitlment. Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * CFINSPEC-506 license execution check Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Software entitlement call changes and handling of error Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Error handling from chef licensing error to standard error Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Updated Gemfile to pick the chef licensing changes from git url Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Trying with https url Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Revert "Trying with https url" This reverts commit 33f1f4c0ecacf9ba2826e25e55b41219903ea736. * Adds the license not entitled exit code Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Updates the require statement Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Fix typo Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Set the licensing configure for entitlement id failure Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Use ChefLicensing::Error instead of standard error Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * REFACTOR Move out licensing cong=fig to its own file and make unconditional Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * FIX TESTS - Add reference to licensing config for places where we call runner explicitly. Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Update inputs_test to use license configuration from utils Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> --------- Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> Co-authored-by: Nikita Mathur <nikita.mathur@chef.io> Co-authored-by: Clinton Wolfe <clintoncwolfe@gmail.com> |
||
|
Vasundhara Jagdale
|
530d2d1ef2
|
CFINSPEC-580 Dependent profiles fix for any scheme of version used in profiles (#6410) (#6468)
* Dependent profiles fix for any scheme of version used in profiles * Dependent profile fix when version is blank in dependent profiles --------- Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Co-authored-by: Nikita Mathur <Nik08@users.noreply.github.com> |
||
|
Vasundhara Jagdale
|
ee35f4ed08
|
CFINSPEC-576 : Fix profile gem dependency loading issue when dependent gem is required inside profile libraries. (#6408) (#6467)
* Test profile consist of custom resource which has a gem dependency * CFINSPEC-576 Fix for profile gem depedency loading issue if depedent gem is required in libraries files of the profile * Fix windows test failure for profile gem dependency --------- Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> |
||
|
Vasundhara Jagdale
|
812001abcc
|
CFINSPEC-542 Bug fix for profiles with dependent profiles (#6377) (#6435)
* Matching semver using regex to determine profile name without version in dsl logic * Testing for selecting control using version in require control dsl * Documentation for using version with profile name when including or selecting controls in a profile * Commenting in dsl library about logic for fetching version and matching it * Comment for regex used to fetch version * Minor improvement for regex variable placement Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Co-authored-by: Nikita Mathur <Nik08@users.noreply.github.com> |
||
|
Nikita Mathur
|
440934ca13
|
Foreport 6369 to inspec-6 Extended file format support for waivers: JSON & CSV only (#6371)
* Revert "Revert "CFINSPEC-240 Extended file format support for waivers (#6193)" (#6366)"
This reverts commit
|
||
|
Vasundhara Jagdale
|
35f8c4829f
|
CFINSPEC-533: Fix undefined method 'summary' for Gem::SourceFetchProblem (NoMethodError) when airgapped (#6337) (#6434)
* This fixes the undefined method 'summary' for #<Gem::SourceFetchProblem:0x000000000139dc70> (NoMethodError) while invoking any inspec command if the custom plugin is installed and if inspec commands are run in an air gap environment. This fix removes an earlier call for getting gem specification from rubygems.org which is the cause for above error and replaces it with fetching the summary from a locally saved gemspec file. This will work in both airgap and non-airgap environments. * Adds test for reading description from local gemspec file. Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> |
||
|
Vasundhara Jagdale
|
4376365304
|
Fix env smoke test by updating ERB.new in inspec env; add additional test (#6342) (#6440)
* Failing test for Ruby 3.1 - ERB.new call options changed * Fix for Ruby 3.1 by using named params to ERB.new * Restrict test to non-windows systems * Restrict test to non-windows systems Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> Co-authored-by: Clinton Wolfe <clintoncwolfe@gmail.com> |
||
|
Nikita Mathur
|
3d54b17049
|
Upgraded mock ubuntu test from 20.04 to 22.04 (#6456)
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> |
||
|
Vasundhara Jagdale
|
1e1e804b87
|
Prevent negative status from crashing launchd service resource (#6262) (#6433)
Signed-off-by: dtheyer <davidheyer2016@gmail.com> Co-authored-by: dtheyer <davidheyer2016@gmail.com> |