Skip to content

Commit e724160

Browse files
authored
Merge pull request #307 from maximillions-dev/fix/ocsp-stapling-display
fix: show OCSP stapling as informational, not pass/fail
2 parents 2c9fb00 + 59a6c70 commit e724160

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

src/client/components/Results/TlsConnection.tsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,8 @@ import Row from 'client/components/Form/Row';
33

44
const yesNo = (v: boolean) => (v ? '✅ Yes' : '❌ No');
55

6+
const ocspStatus = (v: boolean) => (v ? 'Yes' : 'Not enabled');
7+
68
const formatEphemeralKey = (k: any): string => {
79
if (!k?.type) return '';
810
const parts = [k.type];
@@ -28,7 +30,7 @@ const TlsConnectionCard = (props: {
2830
{d.alpnProtocol && <Row lbl="ALPN" val={d.alpnProtocol} />}
2931
<Row lbl="Forward Secrecy" val={yesNo(!!d.forwardSecrecy)} />
3032
<Row lbl="Session Resumption" val={yesNo(!!d.sessionResumption)} />
31-
<Row lbl="OCSP Stapling" val={yesNo(!!d.ocspStapled)} />
33+
<Row lbl="OCSP Stapling" val={ocspStatus(!!d.ocspStapled)} />
3234
<Row
3335
lbl="Certificate Trust"
3436
val={d.authorized ? '✅ Trusted' : `❌ ${d.authError || 'Untrusted'}`}

0 commit comments

Comments
 (0)