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

symbiosis-ssl can generate SSL config for sites that have no certificate #44

Closed
patch0 opened this issue Jun 13, 2017 · 2 comments
Closed

Comments

@patch0
Copy link
Contributor

patch0 commented Jun 13, 2017

symbiosis-ssl can generate SSL config for sites that have no certificate returned by Lets Encrypt. This can lead to invalid configuration, and Apache being unable to re-start.

This has been observed both in terms of missing certs that were never returned successfully from Lets Encrypt, or where symbiosis-ssl didn't have permission to write the certificate, but still wrote the SSL config.

Originally reported on Bytemark's Gitlab by @dedwards on 2016-08-10T11:32:30.849Z

@patch0
Copy link
Contributor Author

patch0 commented Jun 13, 2017

Not sure how this can happen. I'll need more info to investigate further.

At the time LetsEncrypt issues the cert, it is parsed as an X509 certificate. If LetsEncrypt returns a zero-length or otherwise corrupt cert, then this stage will fail.

When reading the certificates from disc, symbiosis-ssl assesses each one for validity, and if the file is empty at this point it should be ignored as an invalid set.

Originally posted by @patch0 on 2016-05-16T12:27:55.000Z

@patch0 patch0 added the bug label Jul 10, 2017
@patch0 patch0 added the invalid label Jul 20, 2017
@patch0
Copy link
Contributor Author

patch0 commented Jul 20, 2017

Having spoken to @dedwards he's not seen this for a while, so I'll close this as invalid.

@patch0 patch0 closed this as completed Jul 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant