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

OpenSSL certificate generation command in secrets quickstart docs is broken #837

Closed
maxalbert opened this issue May 22, 2019 · 0 comments · Fixed by #838
Closed

OpenSSL certificate generation command in secrets quickstart docs is broken #837

maxalbert opened this issue May 22, 2019 · 0 comments · Fixed by #838
Labels
bug Something isn't working deployment docs Documentation issues FlowAPI Issues related to the FlowKit API

Comments

@maxalbert
Copy link
Contributor

The instructions Installation > Secrets Quickstart are slightly buggy and currently don't always allow to spin up a working system. In particular, the command which creates an OpenSSL certificate can fail.

There are two reasons for this. Firstly, it contains the instruction [[ "Darwin" -eq "$(uname -s)" ]]. The eq does a numeric comparison rather than a string comparison, and due to some bash weirdness this leads to the command always succeeding. On a non-Darwin system this means that the configuration file can't be found and as a result the cert.pem file won't be generated. (Which leads to an SSL error when hypercorn attempts to start up inside the container.)

Another issue is that on CentOS the config file does not exist at the location/etc/ssl/openssl.cnf but rather at /etc/pki/tls/openssl.cnf, which equally results in a broken certificate and an SSL error when hypercorn starts up.

@maxalbert maxalbert added bug Something isn't working docs Documentation issues deployment FlowAPI Issues related to the FlowKit API labels May 22, 2019
@maxalbert maxalbert mentioned this issue May 22, 2019
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working deployment docs Documentation issues FlowAPI Issues related to the FlowKit API
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant