Skip to content

Commit

Permalink
fix(1Password): KeyringHelper no longer present in 8.10.28 build
Browse files Browse the repository at this point in the history
  • Loading branch information
scottames committed Mar 21, 2024
1 parent e41cbf9 commit 7c6f507
Showing 1 changed file with 7 additions and 6 deletions.
13 changes: 7 additions & 6 deletions config/scripts/1Password.sh
Original file line number Diff line number Diff line change
Expand Up @@ -67,14 +67,15 @@ chmod 4755 /usr/lib/1Password/chrome-sandbox
# Normal user group GIDs on Fedora are sequential starting
# at 1000, setting to something higher to potentially avoid conflict

HELPER_PATH="/usr/lib/1Password/1Password-KeyringHelper"
BROWSER_SUPPORT_PATH="/usr/lib/1Password/1Password-BrowserSupport"

# Setup the Core App Integration helper binaries with the correct permissions and group
chgrp "${GID_ONEPASSWORD}" "${HELPER_PATH}"
# The binary requires setuid so it may interact with the Kernel keyring facilities
chmod u+s "${HELPER_PATH}"
chmod g+s "${HELPER_PATH}"
# ## KeyringHelper no longer present in 8.10.28 build
# HELPER_PATH="/usr/lib/1Password/1Password-KeyringHelper"
# # Setup the Core App Integration helper binaries with the correct permissions and group
# chgrp "${GID_ONEPASSWORD}" "${HELPER_PATH}"
# # The binary requires setuid so it may interact with the Kernel keyring facilities
# chmod u+s "${HELPER_PATH}"
# chmod g+s "${HELPER_PATH}"

# BrowserSupport binary needs setgid. This gives no extra permissions to the binary.
# It only hardens it against environmental tampering.
Expand Down

0 comments on commit 7c6f507

Please sign in to comment.