Skip to content

Comments

feat/safely add the support of having HTTP w/ self-signed SSL#13

Open
pinotelio wants to merge 1 commit intoparadixe:mainfrom
pinotelio:feat/support-self-signed-ssl
Open

feat/safely add the support of having HTTP w/ self-signed SSL#13
pinotelio wants to merge 1 commit intoparadixe:mainfrom
pinotelio:feat/support-self-signed-ssl

Conversation

@pinotelio
Copy link

@pinotelio pinotelio commented Jan 31, 2026

Overview of this PR

  • Added HTTPS support with self-signed SSL certificate for enhancing the security of the dashboard and administrative tasks.
  • Made self-signed certificate configurable using ENABLE_HTTPS environment variable (maintained the current behaviour and defaulting the wizard setup to HTTP and current behaviour).
  • The default behaviour can be changed to HTTPS later on, as desired.
  • Added SSL certificate library in dashboard app, to auto-generate certificates when necessary based on the environment variable.
  • Updated all the corresponding scripts and templates to properly handle curl commands and the user outputs based on the HTTP/HTTPS conditions
  • Several whitespace/trailing-spaces/indentation enhancements to the modified files

Tests ✅

  • Tested both with and without the self-signed certificate to make sure the feature works as expected and also the default behaviour is not broken.

Here are the self-signed settings test results:

setup-wizard
Setup Wizard ☝

setup-wizard-output
Setup Wizard Output ☝

node-registration
Successful Node Registration ☝

verify-node-in-dashboard
Verified Registered Node in the Dashboard ☝

Deployments Note

  • For my e2e tests, I had to manually build the dashboard container images and temporarily comment out some sections in the setup scripts, so that the scripts point to my own custom image instead of the production image. That being said, if all goes well and the PR gets merged, ghcr.io/paradixe/conduit-dashboard:latest needs to be built with the latest changes and pushed, to make this feature work.

@paradixe
Please feel free give a it a look and additional tests if needed
❤️ 🕊️✌️

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.

1 participant