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

Fix ca-certificates dependency for deb packages #92

Merged
merged 1 commit into from
Jul 21, 2023

Conversation

mattkeenan
Copy link
Contributor

This is a fix for #91. In order to install portmaster files must be downloaded via https, but this requires trusted certificates to be installed or the download will fail because the SSL cert on the server won't be trusted. To fix this we include ca-certificates as a dependency (this package is installed on desktop installs). This will be useful on some containers or iot installs, and to a lesser extent minimal server installs (like debootstrap installs).

@dhaavi
Copy link
Member

dhaavi commented Jul 21, 2023

Please rebase.

@mattkeenan mattkeenan closed this Jul 21, 2023
@mattkeenan mattkeenan force-pushed the fix/issues/91 branch 2 times, most recently from 9210c2d to d57dc41 Compare July 21, 2023 07:26
@mattkeenan mattkeenan reopened this Jul 21, 2023
@mattkeenan
Copy link
Contributor Author

Sorry, I'm far more used to the git command line than github. Rebased to safing/develop.

@dhaavi
Copy link
Member

dhaavi commented Jul 21, 2023

LGTM, merging.

I don't know yet when we will do the next release though - hopefully soon.

Thanks for the fixes!

@dhaavi dhaavi merged commit 815179a into safing:develop Jul 21, 2023
36 checks passed
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