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

Update certificate for macOS build (& fix notarization) #1721

Closed
holmesworcester opened this issue Aug 18, 2023 · 12 comments
Closed

Update certificate for macOS build (& fix notarization) #1721

holmesworcester opened this issue Aug 18, 2023 · 12 comments
Assignees
Labels
bug Something isn't working chore

Comments

@holmesworcester
Copy link
Contributor

From Apple:

Your Distribution Certificate will no longer be valid in 30 days. To generate a new certificate, sign in and visit Certificates, Identifiers & Profiles.

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.

@holmesworcester
Copy link
Contributor Author

@kingalg are you seeing any problems with Gatekeeper warnings when doing a fresh install on Mac OS?

@siepra did this get updated?

@kingalg
Copy link
Collaborator

kingalg commented Nov 15, 2023

@holmesworcester no problems or unexpected messages on my side.

@holmesworcester
Copy link
Contributor Author

holmesworcester commented Nov 16, 2023

Got a report from a user of:

image

Our own devices won't be a reliable view of this, apparently.

@holmesworcester holmesworcester added the bug Something isn't working label Nov 16, 2023
@siepra
Copy link
Contributor

siepra commented Nov 17, 2023

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"

@holmesworcester
Copy link
Contributor Author

holmesworcester commented Nov 17, 2023

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

@siepra
Copy link
Contributor

siepra commented Nov 17, 2023

What version did the user try to run?
Here is a commit of Barteks' for fixing notarization 1d98ec3
It should be fine since version 1.9.6 (2.0.2-alpha.0)

@holmesworcester
Copy link
Contributor Author

holmesworcester commented Nov 17, 2023

There are two separate issues:

  1. Altool deprecation
  2. Certificate expiration

This issue is for the second, which I believe is the problem here.

The user was installing from the website.

@siepra
Copy link
Contributor

siepra commented Nov 17, 2023

I don't think it's the second case as all our certificates are valid

@siepra
Copy link
Contributor

siepra commented Nov 17, 2023

@Kacper-RF spotted an error trying to notarize the app last time (it was easy to miss as it didn't interrupt the job).
We pushed a fix for configuration and started the build https://github.com/TryQuiet/quiet/pull/2078/files

Image

@holmesworcester
Copy link
Contributor Author

holmesworcester commented Nov 17, 2023

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?

@siepra
Copy link
Contributor

siepra commented Nov 17, 2023

Does every release involve getting a new certificate? - no, a certificate have it's expiration date after which it must be renewed

@siepra
Copy link
Contributor

siepra commented Nov 17, 2023

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

@siepra siepra changed the title Update certificate for macOS build Update certificate for macOS build (fix notarization) Nov 17, 2023
@siepra siepra changed the title Update certificate for macOS build (fix notarization) Update certificate for macOS build (& fix notarization) Nov 17, 2023
@siepra siepra self-assigned this Nov 20, 2023
@siepra siepra closed this as completed Jan 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working chore
Projects
Archived in project
Development

No branches or pull requests

3 participants