ssl-cert-check icon indicating copy to clipboard operation
ssl-cert-check copied to clipboard

Published 20 hours ago • verified publisher icon Matty9191

parsing incorrectly

Open copenhaus opened this issue 4 years ago • 4 comments

Hi, I'd like to report that whenever the Issuer has the character, comma, in its name, the output failed to parse the Issuer correctly.

As a sample: https://imgur.com/a/aq5kufb

Thanks...

copenhaus avatar Nov 25 '20 08:11 copenhaus

Yeah, I'm also faced with this, for example: ssl-cert-check -s acme.com/page -p 443

Host Status Expires Days

ERROR: The file named /var/tmp/cert.X0uptO is unreadable or doesn't exist ERROR: Please check to make sure the certificate for acme.com/page:443 is valid

woraccmsq avatar Dec 01 '20 19:12 woraccmsq

After update to 4.14: ssl-cert-check -s acme.com/page -p 443

Host Status Expires Days

acme.com/page:443 Unable to resolve the DNS name acme.com/page Unknown 0, 0 0

woraccmsq avatar Dec 01 '20 20:12 woraccmsq

ERROR: The file named /var/tmp/cert.kK1m3s is unreadable or doesn't exist,ERROR: Please check to make sure the certificate for somedomain-name.net:443 is valid

Instead of showing this error, can we return user friendly msg e.g UNKNOWN or NA or INVALID CERTS

Because of this error, my whole dashboard is breaking

sandykadam avatar Apr 19 '21 13:04 sandykadam

All,

1) Attached is a patch which fixes the original "Certificate Issuer with a comma" bug

2a) acme.com/page is an invalid hostname; use acme.com 2b) -p 443 is unnecessary, since that is the default

Enjoy! Daniel Lewart Urbana, Illinois

[certissuer.txt](https://github.com/Matty9191/ssl-cert-check/files/7896453/certissuer.txt

#115

lewart3 avatar Jan 19 '22 11:01 lewart3