Skip to content

Commit

Permalink
Merge branch 'drop-display-cn' of ssh://github.com/TinCanTech/easy-rs…
Browse files Browse the repository at this point in the history
…a into TinCanTech-drop-display-cn

Signed-off-by: Richard T Bonhomme <[email protected]>
  • Loading branch information
TinCanTech committed Apr 10, 2024
2 parents 6afbc29 + fac93ba commit f5abd51
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 83 deletions.
1 change: 1 addition & 0 deletions ChangeLog
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ Easy-RSA 3 ChangeLog

3.2.0 (TBD)

* Remove command and function display_cn(), unused (be8f400) (#1114)
* Introduce Options to edit Request Subject during command 'sign-req'
Global Option: --new-subject -- Command 'sign-req' option: 'newsubj'
First proposed in: (#439) -- Completed: (83b81c7) (#1111)
Expand Down
87 changes: 4 additions & 83 deletions easyrsa3/easyrsa
Original file line number Diff line number Diff line change
Expand Up @@ -499,9 +499,6 @@ These commands are safe to test and will NOT effect your PKI.
Check <SERIAL> number is unique:
serial|check-serial <SERIAL>

Display CN of request or certificate: <form> = req|x509
display-cn <form> <DIR/FILE_NAME>

Display DN of request or certificate: <form> = req|x509
display-dn <form> <DIR/FILE_NAME>

Expand Down Expand Up @@ -616,11 +613,12 @@ Certificate & Request options: (these impact cert/req field values)
--copy-ext : Copy included request X509 extensions (namely subjAltName)
For more info, see: 'easyrsa help copyext'

--san|--subject-alt-name=<subjectAltName>
: Add a subjectAltName.
--san|--subject-alt-name=SUBJECT_ALT_NAME
: Add a subjectAltName. Can be used multiple times.
For more info and syntax, see: 'easyrsa help altname'

--new-subject : Specify a new subject field to sign a request with.
--new-subject='SUBJECT'
: Specify a new subject field to sign a request with.
For more info and syntax, see: 'easyrsa help subject'

--usefn=NAME : export-p12, set 'friendlyName' to NAME
Expand Down Expand Up @@ -3393,79 +3391,6 @@ update_db() {
die "Failed to perform update-db."
} # => update_db()

# Display commonName
display_cn() {
format="$1"
path="$2"
var_name="$3"
shift "$#"

case "$format" in
req)
def_dir=reqs
dot_3=req
;;
x509)
def_dir=issued
dot_3=crt
;;
'')
user_error "display_cn - Unspecified format"
;;
*)
user_error "display_cn - Unknown format: '$format'"
esac

# Check for absolute or relative file name
if [ -e "$path" ]; then
: # ok
else
in_file="${EASYRSA_PKI}/${def_dir}/${path}.${dot_3}"
in_file_1="${EASYRSA_PKI}/${def_dir}/${path}"
if [ -e "$in_file" ]; then
path="$in_file"
elif [ -e "$in_file_1" ]; then
path="$in_file_1"
else
user_error "\
display_cn - Type '$format', missing: '$path'"
fi
fi

# Extract commonName
error_info="Certs require form 'x509'; Reqs require form 'req'"
if ssl_cn="$(
export OPENSSL_CONF=/dev/null
"$EASYRSA_OPENSSL" "$format" -in "$path" -noout -subject \
-nameopt utf8,sep_multiline,space_eq,lname,align | \
grep '^[[:blank:]]*commonName'
)"
then
unset -v error_info
else
die "display_cn - ssl_cn: Not found 'commonName'"
fi

# Return commonName
if [ "$internal_batch" ] || [ "$EASYRSA_BATCH" ]; then
# Remove the label
ssl_cn="${ssl_cn#*= }"
# if set then return CN via variable
if [ "$var_name" ]; then
verbose "display_cn - force_set_var '$var_name' '$ssl_cn'"
force_set_var "$var_name" "$ssl_cn" || \
die "display_cn - force_set_var '$var_name' '$ssl_cn'"
else
print "$ssl_cn"
fi
else
print "$ssl_cn"
fi

unset -v ssl_cn var_name in_file in_file_1 \
def_dir dot_3 path format internal_batch
} # => display_cn()

# display cert DN info on a req/X509, passed by full pathname
display_dn() {
[ "$#" = 2 ] || die "\
Expand Down Expand Up @@ -5533,10 +5458,6 @@ Place a copy of easyrsa-tools.lib in a standard system location."
check_serial_unique "$@" || \
easyrsa_exit_with_error=1
;;
display-cn)
verify_working_env
display_cn "$@"
;;
display-dn)
verify_working_env
display_dn "$@"
Expand Down

0 comments on commit f5abd51

Please sign in to comment.