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.

Sign up for GitHub

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

Jump to bottom

make-certs: Handle missing OpenSSL installation #595

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

make-certs: Handle missing OpenSSL installation #595

wants to merge 1 commit into from

Commits on Aug 1, 2023

  1. make-certs: Handle missing OpenSSL installation 

    If there is no OpenSSL installation on the system, where the
`make-certs` script is run at, no error message will be shown; instead
the script will fail silently and no certificates will be generated.

This change introduces a simple check, if the `openssl` binary is
present and informs the user, if it's missing, shortening debugging time
from minutes to mere seconds.

A bashism has been used so a pretty message gets printed without moving
the check to a section before `set -e`.

Signed-off-by: Kamil Aronowski <kamil.aronowski@yahoo.com>
    @aronowski
    aronowski committed Aug 1, 2023
    Configuration menu
    Copy the full SHA
    8beb4ef View commit details
    Browse the repository at this point in the history