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

1Password recipes still pulling down v7 when v8 is out (though v7 updates are still coming) #222

Open
fseesink opened this issue May 25, 2022 · 5 comments

Comments

@fseesink
Copy link

Subject line covers it.

Not sure what best approach is, but figured I would mention that 1Password v8 has been out for a short bit now. The current 1Password recipes are still aimed at v7. They did help to pull down the latest v7 patch on 18 May as the history shows.

Unfortunately, it does not look like you can just tweak the current URL in the download recipe from

https://app-updates.agilebits.com/download/OPM7

to

https://app-updates.agilebits.com/download/OPM8

as that doesn't exist.

Best I have found this page where they have a direct link to

https://downloads.1password.com/mac/1Password.zip

if that helps.

@mtconleyuk
Copy link

Given that 1Password 8 is Electron-based and requires a subscription, whereas 1Password 7 is native code and still supports people who are grandfathered in without subscriptions and with local vaults, it might make sense to have a separate recipe for handling version 8.

@jannheider
Copy link

We should leave this recipe as it is as there are still might coming some v7 patches and v7 does still have standalone (non-subscription) licenses

AgileBits renamed the App from "1Password 7" to "1Password"

I made new "1Password latest" recipe as we needed to Deploy v8
Take a look at jannheider-recipes

@basdeg
Copy link

basdeg commented Jun 8, 2022

i wrote a little sloppy pre-flight script for the 1Password 7 recipe to prevent overwriting of manual 1Password 8 installations;

#!/bin/zsh

# ONLY FOR 1PW 7 Munki implementations!!
# this script is a must check if there is a 1Password 8 installation, if so, then exit 1.


# Find all 1Password Apps in Applications folders
ITEMPATHS=("${(@f)$(mdfind -onlyin /Applications -name "1Password")}")
echo "$ITEMPATHS"

# now a for loop to loop through the array items
for i in $ITEMPATHS; do
echo $i

# check if this is a zipFile, then its probberly a backup file, the installation must abort before it gets deleted!
filename=$(basename -- "$i")
extension="${filename##*.}"

if [ "$extension" = "zip" ]; then
  echo "found a zip file, this file has no version and will probberly removed with the postinstall script"
  break 2
else
  echo "not a zip, continue in checking versions"
fi

# retrieve App version
APPVERSION=$(defaults read $i/Contents/Info CFBundleShortVersionString)
echo $APPVERSION

# now compare version
# Note: when used in a script, you will probably have to autoload is-at-least before using it. In an interactive shell, it is often already loaded, because many other autoloader functions will have already loaded it.
autoload is-at-least
if is-at-least 8.0 $APPVERSION; then
  echo "This found version is higher than v8.0, abort installation, uninstall this first before installing 1PW 7"
  exit 1
else
  echo "This version is lower than version 8, no problem yet!"
fi

; done

exit 0

@mpanighetti
Copy link
Contributor

I had a PR to update these recipes to work with 1Password 8+, but closed it since others have made recipes that accomplish this. I'd recommend this recipe be updated to specify that it only applies to 1Password 7.

@mtconleyuk
Copy link

I think keeping it restricted to 1Password 7 is a fine idea; that's what we install here, as we have no policy in place to deal with the version 8 subscription mechanism.

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

No branches or pull requests

5 participants