Skip to content
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

Perform Gatekeeper scan to pre-warm app launch #2505

Merged
merged 3 commits into from
Feb 18, 2024
Merged

Perform Gatekeeper scan to pre-warm app launch #2505

merged 3 commits into from
Feb 18, 2024

Conversation

zorgiepoo
Copy link
Member

@zorgiepoo zorgiepoo commented Feb 18, 2024

This avoids users seeing a "Verifying..." Dialog when the installed update is (re-)launched.

This scan is only done for macOS 14.4+ onwards (gktool was introduced in macOS 14.0 but had issues).

Also this scan is only done if Autoupdate's team identifier matches the new update's team identifier. Otherwise the OS may think Autoupdate is modifying a bundle that it shouldn't be permitted to.

Fixes #2491

Misc Checklist

  • My change requires a documentation update on Sparkle's website repository
  • My change requires changes to generate_appcast, generate_keys, or sign_update

Testing

I tested and verified my change by using one or multiple of these methods:

  • Sparkle Test App
  • Unit Tests
  • My own app
  • Other (please specify)

Tested updating my own notarized app and saw gktool run without issue when Autoupdate & app bundle to be updated are signed with same Team Identifier. Tested gktool is not ran when Team Identifier differs or system is running an OS before 14.4.

macOS version tested:
14.4 Beta (23E5196e)
14.3 (23D56) (skipped)

This avoids users being a "Verifying..." Dialog when the installed update is (re-)launched.

This scan is only done for macOS 14.4+ onwards (gktool was introduced in macOS 14.0 but had issues).

Also this scan is only done if Autoupdate's team identifier matches the new update's team identifier. Otherwise the OS may think Autoupdate is modifying a bundle which it shouldn't be permitted to (this is a bug).
@zorgiepoo zorgiepoo added this to the 2.6 milestone Feb 18, 2024
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.

"Verifying..." Dialog Box when app is restarted for an update
1 participant