Skip to content
This repository has been archived by the owner on Jun 1, 2020. It is now read-only.

Allow the use of a separate ca chain file #26

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

duncan-brown
Copy link
Contributor

This pull request implements a HTTPS_CHAIN_FILE environment variable that (optionally) allows the user to specify a CA chain. If this is not given then HTTPS_CERT_FILE must contain the full CA chain as well as the certificate, as before.

The motivation is because https://github.com/linuxserver/docker-letsencrypt generates separate chain and cert files.

@duncan-brown duncan-brown changed the title Allow the user of a separate ca chain file Allow the use of a separate ca chain file Nov 14, 2019
@duncan-brown duncan-brown force-pushed the cert-chain branch 2 times, most recently from 93fe871 to d85540d Compare November 14, 2019 15:13
@duncan-brown
Copy link
Contributor Author

@skoranda sorry for the force pushes, I fat-fingered my git repo.

I've tested this with my comanage instance and it works with a stack that looks like this.

@duncan-brown
Copy link
Contributor Author

@skoranda poke on this pull request.

@skoranda
Copy link
Contributor

I believe that the image/container you point to also generates the file(s) fullchain.pem, just like the standard 'certbot' utility does. That file contains the end entity certificate and the chain together in one file.

Can you use that file instead of needing this patch?

The trend for all webservers (including Apache HTTP Server) is to have them all in one file now and I would like to follow that trend if possible.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants