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: use selfsigned instead of devcert for certificate generation #1650

Merged
merged 3 commits into from
Nov 2, 2023

Conversation

KEGustafsson
Copy link
Contributor

@KEGustafsson KEGustafsson commented Oct 20, 2023

Fixes #1649 seen with Devcert self-signed certificate generation tool. Devcert require sudo for certain operations and that is not available from docker.

Devcert replaced with Selfsigned package
@KEGustafsson KEGustafsson changed the title Selfsigned certificate generation to fix Devcert issue #1649 Selfsigned certificate generation to fix Devcert issue Oct 20, 2023
@tkurki tkurki added the fix label Oct 22, 2023
@tkurki
Copy link
Member

tkurki commented Oct 22, 2023

@cmotelet how does this look to you? Will this work on windows?

@tkurki tkurki changed the title Selfsigned certificate generation to fix Devcert issue Uae selfsigned instead of devcert for certificate generation Oct 22, 2023
@tkurki tkurki changed the title Uae selfsigned instead of devcert for certificate generation fix: use selfsigned instead of devcert for certificate generation Oct 22, 2023
@KEGustafsson
Copy link
Contributor Author

KEGustafsson commented Oct 23, 2023

I tested selfsigned image on Windows 10 with Docker Desktop. Docker-compose file applied from Cmd and WSL2. SK container came up fine.

package.json Outdated Show resolved Hide resolved
@tkurki tkurki merged commit 2761003 into SignalK:master Nov 2, 2023
2 checks passed
@cmotelet
Copy link
Member

@cmotelet how does this look to you? Will this work on windows?

@tkurki , Sorry I missed this discussion !
Since August 25, 2023 in the Windows installer version 1.1.0, I automatically generate a self-signed certificate valid for 10 years.
https://github.com/SignalK/signalk-server-windows/blob/master/NSIS/signalk-server.nsi#L359
I've just validated version 2.4.1 of the Signal K server and the new server-integrated generation with selfsign of the certificate also works perfectly under Windows.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants