search

google / macops-keychainminder

SecurityAgentPlugin for monitoring keychain password synchronization
94 stars 18 forks source link

Update uninstall.sh #8

Closed howiekaye closed 8 years ago

howiekaye commented 8 years ago

screensaver behavior did not revert to the new UI when I disabled keychainminder. Re-running the update_authdb.py script with remove --restore-screensaver seemed to be needed.

russellhancox commented 8 years ago

That is the intended behavior. While changing to the old UI when installing is necessary (if not already done) to make KeychainMinder work, we don't know whether the old or new UI was in use at install-time.

The update_authdb.py script has the flag so organizations can update uninstall.sh as they wish, I think the current behavior is the safest default.

howiekaye commented 8 years ago

I disabled our keychainminder knob (in gMenu/Preferences), but did not get back the old screensaver (though the text in gMenu says it should). The uninstall script deletes itself, so I had to find the security commands to run manually....

On Thu, Feb 25, 2016 at 1:06 PM, Russell Hancox notifications@github.com wrote:

That is the intended behavior. While changing to the old UI when installing is necessary (if not already done) to make KeychainMinder work, we don't know whether the old or new UI was in use at install-time.

The update_authdb.py script has the flag so organizations can update uninstall.sh as they wish, I think the current behavior is the safest default.

— Reply to this email directly or view it on GitHub https://github.com/google/macops-keychainminder/pull/8#issuecomment-188907043 .

russellhancox commented 8 years ago

Add a flag to uninstall.sh to pass --restore-screensaver when set and we can use that.