OpenVPN / easy-rsa

easy-rsa - Simple shell based CA utility
Other
3.96k stars 1.18k forks source link

Remove obsolete 'sed' expansion of openssl-easyrsa.cnf #1147

Closed TinCanTech closed 4 weeks ago

TinCanTech commented 1 month ago

Removed:

All expansion is now performed by expanding here-docs.

TinCanTech commented 1 month ago

Unfortunately, this does not take into account use of ~LibreSSL~ ANY SSL with a custom openssl-easyrsa.cnf file. It forcibly over writes the user SSL config.

This means that removing sed expansion is not possible.

Alternatively, the code can catch this specific circumstance and refuse it with an informative error message. ~eg. EasyRRSA does not support custom SSL config for use with LibreSSL~

I'm looking hard at the alternative option above..


This is an ideal candidate for easyrsa-tools.lib, how often is a user going to customise openssl-easyrsa.cnf ? [Applies to both OpenSSL and LibreSSL]

Linking: #1121

Closed for the reason given in #1116