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

Updated dockerfile with options for including your own root certificate and deploying your container to an Azure App Service. #269

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jcbartle
Copy link

@jcbartle jcbartle commented Dec 1, 2023

While deploying this privately, I encountered a few issues which caused some significant trouble. Two of these are addressed by updating the dockerfile, and I hope this PR can save others some time and headaches.

My modifications are commented out by default and are opt-in configuration decisions.

Using a Proxy / SWG / etc. which intercepts SSL

This scenario requires uploading your own root certificate to the container and configuring Node to trust it. This modification was based on this Stack Overflow conversation.

Custom Container + Azure App Service

Our solution used the above custom container deployed to an Azure App Service via an Azure Container Registry. The container would continually get shut down due to failing to respond to HTTP ping requests. Based on this somewhat related tutorial, I tried switching the HOSTNAME environment variable value to 0.0.0.0. This instantly solved the problem.

…te and deploying your container to an Azure App Service.
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