Skip to content

Fix updating not working the first time if bundle ID ends with ".app"#2882

Merged
zorgiepoo merged 1 commit into
2.xfrom
push-lkvzztvoxsmx
Jun 6, 2026
Merged

Fix updating not working the first time if bundle ID ends with ".app"#2882
zorgiepoo merged 1 commit into
2.xfrom
push-lkvzztvoxsmx

Conversation

@zorgiepoo

Copy link
Copy Markdown
Member

Use a different app cache directory if bundle ID ends with ".app" so that the cache directory Sparkle writes into does not look like an app bundle and cause other systematic issues. This can occur if a developer uses an ill-formed bundle identifier rather than conventional reverse-DNS one.

Fixes issue brought up in discussion #2881

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)

Created a test app that has a bundle ID ending with ".app" and notarized it.
Confirmed "app modification prompt" and installation failure issue occurs without this fix, and that it is resolved with this change. After attempting to reproduce, I removed the app cache directory in ~/Library/Caches/, removed the app in Settings > Privacy & Security > App Management.

macOS version tested: 26.5 (25F71)

Use a different app cache directory if bundle ID ends with ".app" so that the cache directory Sparkle writes into does not look like an app bundle and cause other systematic issues. This can occur if a developer uses an ill-formed bundle identifier rather than conventional reverse-DNS one.
@zorgiepoo zorgiepoo added this to the 2.9.3 milestone Jun 4, 2026
@zorgiepoo zorgiepoo merged commit e6a344a into 2.x Jun 6, 2026
2 checks passed
@zorgiepoo zorgiepoo deleted the push-lkvzztvoxsmx branch June 6, 2026 01:45
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.

1 participant