-
Notifications
You must be signed in to change notification settings - Fork 30
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
Use Github Releases for hosting and fetching update files. Enables our fork of Atom to update in-app. #86
Comments
We should look into some higher-level abstractions already available to do this kind of thing:
|
Actually, I think the requirement to have signed apps is just so we don't get "this app is unauthorized" warnings, and so we don't have our updater stopped with a warning by the OS. (See this documentation.) We won't get far attempting auto-updating without code signing. (I assume Maybe we should disable auto-updating (which is basically broken on this fork until we code sign), and instead show users the GitHub Releases page URL. For example: we could use the GitHub API, check for updates, and just display a URL to download the new version in "About Atom". And perhaps display a bubble notification instead of running the auto-update mechanism. |
We will get this kind of error in the first installation, and it will happen again during updating. So I don't think codesign is a determining factor here. |
I think seeing this kind of errors at all is an issue, right? Many users aren't technical enough to understand what the error means or why it's happening. And their antivirus quarantines the update file. |
This will also be a significant issue for programmers working on work machines as most help desk teams will require only verified software be used, with proper signings for the purposes of auditing |
For Windows, see: https://github.com/Squirrel/Squirrel.Windows/blob/develop/docs/using/github.md
There are no separate docs files I could see for macOS, but here is the repo: https://github.com/Squirrel/Squirrel.Mac
The text was updated successfully, but these errors were encountered: