Skip to content

Commit

Permalink
Add error log for invalid OCSP response (#9674)
Browse files Browse the repository at this point in the history
* Add error log for invalid OCSP response

* Check times only if status is available
  • Loading branch information
maskit authored May 10, 2023
1 parent 29d8737 commit c5b935c
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion iocore/net/OCSPStapling.cc
Original file line number Diff line number Diff line change
Expand Up @@ -958,7 +958,10 @@ stapling_check_response(certinfo *cinf, TS_OCSP_RESPONSE *rsp)
// If ID not present just pass it back to client
Error("stapling_check_response: certificate ID not present in response for %s", cinf->certname);
} else {
TS_OCSP_check_validity(thisupd, nextupd, 300, -1);
if (!TS_OCSP_check_validity(thisupd, nextupd, 300, -1)) {
// The check is just for logging and pass the response back to client anyway
Error("stapling_check_response: status in response for %s is not valid already/yet", cinf->certname);
}
}

switch (status) {
Expand Down

0 comments on commit c5b935c

Please sign in to comment.