* fixing my own stupidity - I am sure that it is just temporary.
@@ -964,7 +964,7 @@ apr_status_t md_cert_read_http(md_cert_t **pcert, apr_pool_t *p,
apr_status_t rv;
ct = apr_table_get(res->headers, "Content-Type");
- if (!res->body || !ct || strcmp("XXapplication/pkix-cert", ct)) {
+ if (!res->body || !ct || strcmp("application/pkix-cert", ct)) {
rv = APR_ENOENT;
goto out;
}
@@ -984,7 +984,8 @@ apr_status_t md_cert_read_http(md_cert_t **pcert, apr_pool_t *p,
else {
cert = md_cert_make(p, x509);
rv = APR_SUCCESS;
- md_log_perror(MD_LOG_MARK, MD_LOG_TRACE3, rv, p, "cert parsed");
+ md_log_perror(MD_LOG_MARK, MD_LOG_TRACE2, rv, p,
+ "parsing cert from content-type=%s, content-length=%ld", ct, (long)data_len);
}
}
}
@@ -1016,7 +1017,7 @@ apr_status_t md_cert_chain_read_http(struct apr_array_header_t *chain,
goto out;
}
else if (!strcmp("application/pem-certificate-chain", ct)
- || (strncmp("text/plain", ct, sizeof("text/plain")-1))) {
+ || !strncmp("text/plain", ct, sizeof("text/plain")-1)) {
/* Some servers seem to think 'text/plain' is sufficient, see #232 */
if (APR_SUCCESS == (rv = apr_brigade_pflatten(res->body, &data, &data_len, res->req->pool))) {
int added = 0;
@@ -1035,7 +1036,8 @@ apr_status_t md_cert_chain_read_http(struct apr_array_header_t *chain,
rv = APR_SUCCESS;
}
}
- md_log_perror(MD_LOG_MARK, MD_LOG_TRACE3, rv, p, "cert parsed");
+ md_log_perror(MD_LOG_MARK, MD_LOG_TRACE2, rv, p,
+ "parsing cert from content-type=%s, content-length=%ld", ct, (long)data_len);
}
else if (!strcmp("application/pkix-cert", ct)) {
md_cert_t *cert;
