Skip to content

Fix MacOS login items showing ambigious name#11373

Merged
phoerious merged 1 commit intokeepassxreboot:developfrom
vallode:fix/8818
Oct 17, 2024
Merged

Fix MacOS login items showing ambigious name#11373
phoerious merged 1 commit intokeepassxreboot:developfrom
vallode:fix/8818

Conversation

@vallode
Copy link
Copy Markdown
Contributor

@vallode vallode commented Oct 16, 2024

Fixes #8818.

Adding the AssociatedBundleIdentifiers plist property mentioned by user @nodeful fixes the issue of an ambigious (to most users) name appearing in their login items menu on MacOS.

Documentation relating to this property can be found here: https://developer.apple.com/documentation/servicemanagement/updating-helper-executables-from-earlier-versions-of-macos#Connect-services-to-app-names-in-System-Settings

Additionally we also add the Program property using the full path of the application bundle, this seems to further prevent the "organisation name" (from the certificate signing) from appearing in the login items screen. Without this extra change Janek Bevendorff's name would still appear after a delay

Screenshots

Before change:
image
After change:
image

Testing strategy

Manually editing the installed launchd.plist file org.keepassxc.KeePassXC.plist in ~/Library/LaunchAgents. I couldn't get the latest KeePassXC develop commit to build on my local machine.

Type of change

  • ✅ Bug fix (non-breaking change that fixes an issue)

@droidmonkey
Copy link
Copy Markdown
Member

Sweet!

@droidmonkey droidmonkey added this to the v2.7.10 milestone Oct 16, 2024
@droidmonkey droidmonkey requested a review from phoerious October 16, 2024 18:56
Copy link
Copy Markdown
Member

@phoerious phoerious left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot! I've been looking for this. That comment at the end of the issue report escaped my attention.

@phoerious phoerious merged commit f07db03 into keepassxreboot:develop Oct 17, 2024
@vallode vallode deleted the fix/8818 branch October 17, 2024 13:59
@patillacode
Copy link
Copy Markdown

patillacode commented Nov 28, 2024

Coming from the closed thread #8818

JFYI this is still happening on macOs Sequoia

Screenshot 2024-11-28 at 7 07 03 PM

@droidmonkey
Copy link
Copy Markdown
Member

This isn't in a release yet

@patillacode
Copy link
Copy Markdown

That makes sense, I didn't check the last release date or if this was in it.

Thanks!

@droidmonkey droidmonkey added the pr: backported Pull request backported to previous release label Dec 22, 2024
pull bot pushed a commit to Graysonbarton/keepassxc that referenced this pull request Jan 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

platform: macOS pr: backported Pull request backported to previous release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"Allow in background" Login items in macOS Ventura, change name

4 participants