Closed embeddinglayer closed 4 months ago
Ideally the response object contains more data for the underlying connection such as CURLINFO_CERTINFO and CURLINFO_TLS_SESSION
Interesting idea! However, it seems that there is no such const as CURLINFO_NUM_CERTS
.
Interesting idea! However, it seems that there is no such const as
CURLINFO_NUM_CERTS
.
Yeah, that code was unintentional looks like vs-code hadn't persisted to disk when I committed. Just removed it, I'll push the full code for certs in a later commit.
We need one or two unittests to make sure that this patch works as expected. Feel free to add them now or leave it there and I will add them later.
Ideally the response object should include the established server ip. Curl already offers that which is a better alternative to opening an entirely new socket connection to resolve that data.