Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix syntax issues #61

Merged
merged 21 commits into from
Sep 10, 2024
Merged

fix syntax issues #61

merged 21 commits into from
Sep 10, 2024

Conversation

jrmetzger
Copy link
Contributor

No description provided.

@jrmetzger jrmetzger requested a review from wdower August 29, 2024 12:59
@jrmetzger jrmetzger added the bug Something isn't working label Aug 29, 2024
@jrmetzger jrmetzger self-assigned this Aug 29, 2024
inspec.yml Outdated Show resolved Hide resolved
jrmetzger and others added 2 commits August 29, 2024 18:27
* replaced usage of "permissions" with "mode" when appropriate

Signed-off-by: kemley76 <[email protected]>

* simplified SV-257926

Signed-off-by: kemley76 <[email protected]>

* standardized use of "exempt" vs "exemption" in various inputs

* fix missed variable rename

Signed-off-by: kemley76 <[email protected]>

* remove a debugging test

Signed-off-by: kemley76 <[email protected]>

* standardize format to be exempt_<thing that is exempt>

Signed-off-by: Kaden Emley <[email protected]>

---------

Signed-off-by: kemley76 <[email protected]>
Signed-off-by: Kaden Emley <[email protected]>
Co-authored-by: suhanimalhotra <[email protected]>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

look into file once first

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

leave variable not hard coded

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test logic double test this

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

plural works in rhel9, verified

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

want the server config

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we want opensshserver

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is good

controls/SV-258116.rb Outdated Show resolved Hide resolved
controls/SV-258230.rb Outdated Show resolved Hide resolved
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Check for approved as well. enabled =

@wdower wdower merged commit dc7b47d into main Sep 10, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants