Skip to content

Commit f7acaa8

Browse files
committed
Change exception for removed root certificates into easy edit multi-value regular expression for Organization name and making it clear that CA's are actively removed from 1+ root stores.
1 parent d01ded1 commit f7acaa8

1 file changed

Lines changed: 6 additions & 4 deletions

File tree

testssl.sh

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8786,10 +8786,12 @@ certificate_info() {
87868786

87878787
out "$indent"; pr_bold " Chain of trust"; out " "
87888788
jsonID="cert_chain_of_trust"
8789-
if [[ "$issuer_O" =~ StartCom ]] || [[ "$issuer_O" =~ WoSign ]] || [[ "$issuer_CN" =~ StartCom ]] || [[ "$issuer_CN" =~ WoSign ]]; then
8790-
# Shortcut for this special case here.
8791-
pr_italic "WoSign/StartCom"; out " are " ; prln_svrty_critical "not trusted anymore (NOT ok)"
8792-
fileout "${jsonID}${json_postfix}" "CRITICAL" "Issuer not trusted anymore (WoSign/StartCom)"
8789+
# Looks for CA's that have their trust removed by the first part of their Organization Name, add multiple with ^(TrustCor Systems|WoSign) etc.
8790+
if [[ "$issuer_O" =~ ^(TrustCor Systems) ]]; then
8791+
# Shortcut for this special case here. There is a difference between not being in a root store and being removed from a root store.
8792+
pr_italic "$issuer_O"; out " is " ; prln_svrty_critical "actively removed from one or more root stores (NOT ok)"
8793+
fileout "${jsonID}${json_postfix}" "CRITICAL" "Issuer removed from one or more root stores ($issuer_O)"
8794+
set_grade_cap "T" "Untrusted certificate chain"
87938795
else
87948796
# Also handles fileout, keep error if happened
87958797
determine_trust "$jsonID" "$json_postfix" || ((ret++))

0 commit comments

Comments
 (0)