-
-
Notifications
You must be signed in to change notification settings - Fork 13
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
Operations fail on /Applications due to lack of permissions. #9
Comments
Yeah, there's no error handling there, I'm not sure I even used that function much myself. This was my first Mac app so there's probably some things missing 😂 I'll take a look tomorrow. |
You did a good job considering that! |
@ivansavra1 would you be able to test this build out for me? If it still doesn't work, I added some debug logging. Launch the app from Terminal by running: This should show logging in the terminal window when it launches. Then drop one of the apps that don't work to remove from quarantine and see what errors it spits out. |
No reply, closing this out. |
Hi, thanks for making Sentinel, it's a pretty useful app, I wanted to suggest you to look into error handling and sudo permissions. When you try to act on a file that's on /Applications (And other system directories), since Sentinel lacks permission to modify files there, the operations fail but the "App was removed from quarantine / signed" message still shows, it seems there is no error handling, and it automatically assumes it was successful after running the command or call.
I know this is the case, because I run
xattr "/Applications/something.app"
and I can see that the quarantine attribute is still there, so in addition to the error handling, there should be a fallback that uses sudo (Getting the correct permission using Touch ID / password) so it works in more cases.Thanks.
The text was updated successfully, but these errors were encountered: