Skip to content

Add USBGuard configuration support, remove configuration support and add verification#68

Merged
amstewart merged 3 commits intoni:masterfrom
texasaggie97:dev/texasaggie97/add-usbguard-verify
Aug 15, 2025
Merged

Add USBGuard configuration support, remove configuration support and add verification#68
amstewart merged 3 commits intoni:masterfrom
texasaggie97:dev/texasaggie97/add-usbguard-verify

Conversation

@texasaggie97
Copy link
Copy Markdown
Collaborator

@texasaggie97 texasaggie97 commented Aug 6, 2025

Summary of Changes

  • Enable usbguard_config
  • Remove configuration since it must be installed and configured manually
  • Implement verification
    • Only verify when usbguard is installed
    • Verify /etc/usbguard/rules.conf exists and is not empty
  • Change EqualsDelimitedConfigFile.get() to handle comments
    • This allows use for USBGuard

Justification

AB#3094079

Testing

  • Ran with and without usbguard installed
  • Verified missing, empty and non-empty rules.conf files returned expected results

Procedure

  • This PR: changes user-visible behavior, fixes a bug, or impacts the project's security profile; and so it includes a CHANGELOG note.
  • I certify that the contents of this pull request complies with the Developer Certificate of Origin.

@texasaggie97 texasaggie97 requested review from a team, AlexHearnNI and amstewart as code owners August 6, 2025 18:12
… retrieval logic

Signed-off-by: Mark Silva <mark.silva@emerson.com>
@texasaggie97 texasaggie97 force-pushed the dev/texasaggie97/add-usbguard-verify branch from d85a929 to 9048f03 Compare August 7, 2025 18:44
…add verification

Signed-off-by: Mark Silva <mark.silva@emerson.com>
Signed-off-by: Mark Silva <mark.silva@emerson.com>
@texasaggie97 texasaggie97 force-pushed the dev/texasaggie97/add-usbguard-verify branch from 9048f03 to b84b989 Compare August 7, 2025 21:15
@amstewart amstewart merged commit 13a7e91 into ni:master Aug 15, 2025
5 checks passed
@texasaggie97 texasaggie97 deleted the dev/texasaggie97/add-usbguard-verify branch August 18, 2025 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants