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

Windows GUI: Add online/stub installer #986

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

cjee21
Copy link
Collaborator

@cjee21 cjee21 commented Dec 4, 2024

There is still room for improvement/refinement...

@JeromeMartinez
Copy link
Member

Thank you.
I am thinking to have a stub installer without the version so not rebuilt at each version, and the version would be available online.

@cjee21
Copy link
Collaborator Author

cjee21 commented Dec 4, 2024

Thank you. I am thinking to have a stub installer without the version so not rebuilt at each version, and the version would be available online.

That is what I am thinking too (like other apps [Firefox?] so users can reuse previously downloaded installer too) but that requires a special URL so I cannot do that now right?

@JeromeMartinez
Copy link
Member

but that requires a special URL so I cannot do that now right?

Right. We need to discuss internally about that. I'll let you know.
Also, a stub by version is also good, version is used by some third party websites for specific the possibility to use a specific version, so it would be an addition.

@cjee21
Copy link
Collaborator Author

cjee21 commented Dec 4, 2024

Need find a way to pass language to the 2nd installer if possible. Right now on first time install (not updating) user needs to select language two times and the 2nd installer appears behind the 1st after language selection.

Or simplify the downloader more so that it doesn't have pages.

@cjee21
Copy link
Collaborator Author

cjee21 commented Dec 4, 2024

Simplified... less than 90KiB now and no double installer visible :)

@cjee21 cjee21 force-pushed the NSIS branch 2 times, most recently from 94af824 to c263943 Compare December 4, 2024 16:51
@JeromeMartinez JeromeMartinez merged commit 689454c into MediaArea:master Dec 5, 2024
3 checks passed
@JeromeMartinez
Copy link
Member

@g-maxime more stuff for the version upgrade script.

@cjee21 cjee21 deleted the NSIS branch December 5, 2024 09:25
@cjee21
Copy link
Collaborator Author

cjee21 commented Dec 5, 2024

@g-maxime more stuff for the version upgrade script.

and digitally signing another installer exe.

@cjee21
Copy link
Collaborator Author

cjee21 commented Dec 9, 2024

@JeromeMartinez If we want to make the stub installer more secure, we can add a check that the downloaded installer is digitally signed with the correct cert before executing it. It will require third party plugin such as the one from Mozilla used by Firefox's stub installer or another one.

If the version-less stub is still desired, one possibility is to add a config or ini file containing version and/or URL to the server that can be read by NSIS ConfigRead or ReadINIStr functions.

@JeromeMartinez
Copy link
Member

If the version-less stub is still desired, one possibility is to add a config or ini file containing version and/or URL to the server that can be read by NSIS ConfigRead or ReadINIStr functions.

We have already a lot of places with the version, we'll check in the team where the version-less installer should check that.

@JeromeMartinez
Copy link
Member

and digitally signing another installer exe.

All is fine now, right?

@cjee21
Copy link
Collaborator Author

cjee21 commented Dec 9, 2024

All is fine now, right?

Yes. The only thing is it does not check the downloaded installer for valid digital signature before executing. Not an issue if the https connection is not tampered with and the server is not compromised but Firefox's stub installer does have verification.

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

Successfully merging this pull request may close these issues.

2 participants