Skip to content
This repository was archived by the owner on Jun 6, 2025. It is now read-only.

🧹 Revert the grub audit matchers#191

Merged
atomic111 merged 3 commits intomondoohq:mainfrom
micheelengronne:patch-4
Apr 3, 2023
Merged

🧹 Revert the grub audit matchers#191
atomic111 merged 3 commits intomondoohq:mainfrom
micheelengronne:patch-4

Conversation

@micheelengronne
Copy link
Copy Markdown
Contributor

I would revert the GRUB audit matchers. The current one does not work.

Signed-off-by: Michée lengronne <michee.lengronne@limawi.io>
@atomic111
Copy link
Copy Markdown
Member

@micheelengronne, thanks for the improvement, but i want to include lines.where( _ == /^[^#]/ ), because it removes the comments from the file

Copy link
Copy Markdown
Contributor

@mm-weber mm-weber left a comment

Choose a reason for hiding this comment

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

Hey @micheelengronne, thank you for finding those faulty checks!
Are the proposed changes on your improvements agreeable to you?

Kind regards,
Manuel

Comment thread core/mondoo-linux-security.mql.yaml Outdated
Comment thread core/mondoo-linux-security.mql.yaml Outdated
Comment thread core/mondoo-linux-security.mql.yaml Outdated
Comment thread core/mondoo-linux-security.mql.yaml Outdated
Signed-off-by: Michée lengronne <michee.lengronne@limawi.io>
Signed-off-by: Michée lengronne <michee.lengronne@limawi.io>
@micheelengronne
Copy link
Copy Markdown
Contributor Author

I normally made the changes and the regex work.

@micheelengronne micheelengronne requested a review from mm-weber April 1, 2023 11:16
Copy link
Copy Markdown
Member

@atomic111 atomic111 left a comment

Choose a reason for hiding this comment

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

@micheelengronne, thanks for the improvement

Comment thread core/mondoo-linux-security.mql.yaml Outdated
Comment thread core/mondoo-linux-security.mql.yaml Outdated
Comment thread core/mondoo-linux-security.mql.yaml Outdated
@atomic111 atomic111 removed the request for review from mm-weber April 1, 2023 11:48
@chris-rock chris-rock changed the title Revert the grub audit matchers 🧹 Revert the grub audit matchers Apr 3, 2023
@atomic111 atomic111 merged commit f8c52d6 into mondoohq:main Apr 3, 2023
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants