-
Notifications
You must be signed in to change notification settings - Fork 88
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
Update certificate for macOS build (& fix notarization) #1721
Comments
@holmesworcester no problems or unexpected messages on my side. |
Isn't it just the fact that people download Quiet from source other than App Store? The solution is to click "Show in Finder", then ctrl+click the Quiet icon and choose "open" |
That is a possible workaround but it's not what users typically have to do. Try downloading Spotify or Slack, e.g., and you'll see that this is not necessary, because the app is properly signed. We've been doing this for years as well. When the warning shows, installation is effectively blocked for typical users, who do not know the workaround, so it's a serious bug if it's not signed correctly. Documentation here: https://developer.apple.com/documentation/security/notarizing_macos_software_before_distribution |
What version did the user try to run? |
There are two separate issues:
This issue is for the second, which I believe is the problem here. The user was installing from the website. |
I don't think it's the second case as all our certificates are valid |
@Kacper-RF spotted an error trying to notarize the app last time (it was easy to miss as it didn't interrupt the job). |
What a weird situation. Apple's warning makes it sound like action is required. Does every release involve getting a new certificate? Either way, let's confirm on a fresh account that the app is properly notarized before considering this complete. Also, can we make it so that the build process fails if notarization fails, so that we will definitely notice, rather than releasing a version with broken notarization? |
Does every release involve getting a new certificate? - no, a certificate have it's expiration date after which it must be renewed |
I don't know how to make electron-builder stop in case of notarization fail @EmiM @vinkabuki you're more familiar with electron-builder, do you have any thoughts? EDIT: maybe this will do (getting rid of try catch) #2081 |
From Apple:
I'm pretty sure this is the macOS certificate we use for avoiding warnings on macOS. We should update this ASAP.
Let me know if you need access to this area.
The text was updated successfully, but these errors were encountered: