Skip to content

Added "Scan at Launch" Preference - #33

Open
uthiel wants to merge 1 commit into
objective-see:mainfrom
uthiel:main
Open

Added "Scan at Launch" Preference#33
uthiel wants to merge 1 commit into
objective-see:mainfrom
uthiel:main

Conversation

@uthiel

@uthiel uthiel commented Aug 23, 2024

Copy link
Copy Markdown

Motivation: I'm managing Macs in my circle of friends. I install KnockKnock on every Mac and add it to the list of Login Items. While it is obvious for power users to click the "Start Scan" button, it isn't for regular users. The "Scan at Launch" pref fixes this problem and makes sure regular users Macs will get scanned at least once on every reboot.

@objective-see

Copy link
Copy Markdown
Owner

Ah interesting use case! One idea/question ...would it not be easier to persist KnockKnock, say as a launch agent and have it run via its command line interface?

Pros: No UI (which in this case I think would be a good thing), output can be captured from stdout and send to you/elsewhere for analysis/post-processing, etc etc.

For more info see:

KnockKnock.app/Contents/MacOS/KnockKnock -h

@uthiel

uthiel commented Aug 25, 2024

Copy link
Copy Markdown
Author

No, because that would increase my responsibilities beyond my comfort level. I do not wish to assume the role of system administrator for all of my friends’ Macs. Instead, I simply want to provide them with the necessary tools to ensure the safe usage of their Macs.

@uthiel

uthiel commented Feb 4, 2025

Copy link
Copy Markdown
Author

Is there any possibility that this PR will be merged in the future? At the time of its release, this PR had no conflicts, which would have made it easy to merge and improve user friendliness for less tech-savvy individuals.

I fail to comprehend why this PR hasn’t been merged yet, and it discourages me from contributing further.

Furthermore running via CLI is not an option because this PR targets less tech savvy users.

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