Skip to content

feat: Warn user of exited (failed) service when running ujust audit-secureblue#1184

Merged
RoyalOughtness merged 9 commits intolivefrom
unknown repository
Jul 21, 2025
Merged

feat: Warn user of exited (failed) service when running ujust audit-secureblue#1184
RoyalOughtness merged 9 commits intolivefrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Jul 16, 2025

When ujust audit-secureblue is ran, detect if a system service had exited with failure. If the service had failed, downgrade the status of the audit from PASS to WARN to alert the user.

josh3kd added 2 commits July 16, 2025 22:07
Detect if a service has failed in addition to detecting whether it has been enabled. If the service has failed, warn the user about the failure instead of passing the audit.
@ghost ghost requested a review from RoyalOughtness as a code owner July 16, 2025 14:21
@ghost ghost changed the title [feat] Warn user if a service had failed when running ujust audit-secureblue [feat] Warn user if a service failed when running ujust audit-secureblue Jul 16, 2025
@ghost ghost changed the title [feat] Warn user if a service failed when running ujust audit-secureblue [feat] Warn user of exited (failed) service when running ujust audit-secureblue Jul 16, 2025
HastD

This comment was marked as resolved.

@ghost ghost changed the title [feat] Warn user of exited (failed) service when running ujust audit-secureblue feat: Warn user of exited (failed) service when running ujust audit-secureblue Jul 17, 2025
RoyalOughtness
RoyalOughtness previously approved these changes Jul 19, 2025
@RoyalOughtness
Copy link
Copy Markdown
Collaborator

@josh3kd couple linter errors, otherwise this looks good

@HastD
Copy link
Copy Markdown
Collaborator

HastD commented Jul 19, 2025

Looks like it's just flagging some minor formatting issues. Running ruff format should automatically fix all of those.

@RoyalOughtness
Copy link
Copy Markdown
Collaborator

@josh3kd I'm unable to merge from live because you didn't grant me the permission 😅

@ghost
Copy link
Copy Markdown
Author

ghost commented Jul 21, 2025

@RoyalOughtness Oops, fixed now. Sorry

@RoyalOughtness RoyalOughtness merged commit 676ae2b into secureblue:live Jul 21, 2025
32 of 37 checks passed
@ghost ghost deleted the WarnOnFailureDuringAudit branch July 25, 2025 04:09
RoyalOughtness pushed a commit to RoyalOughtness/secureblue-dev that referenced this pull request Aug 4, 2025
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.

2 participants