[cpp-xmltooling] branch master updated: Log error on debug for non-mandatory transport authn.

Scott Cantor cantor.2 at osu.edu
Fri Oct 12 11:13:32 EDT 2018


This is an automated email from the git hooks/post-receive script.

scantor pushed a commit to branch master
in repository cpp-xmltooling.

View the commit online:
http://git.shibboleth.net/view/?p=cpp-xmltooling.git;a=commit;h=c010a8c2fd175831fc1a6daadc7d320f742e42d5

The following commit(s) were added to refs/heads/master by this push:
       new  c010a8c   Log error on debug for non-mandatory transport authn.
c010a8c is described below

commit c010a8c2fd175831fc1a6daadc7d320f742e42d5
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Fri Oct 12 11:13:24 2018 -0400

    Log error on debug for non-mandatory transport authn.
---
 xmltooling/soap/impl/CURLSOAPTransport.cpp | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/xmltooling/soap/impl/CURLSOAPTransport.cpp b/xmltooling/soap/impl/CURLSOAPTransport.cpp
index a7148aa..6ff2277 100644
--- a/xmltooling/soap/impl/CURLSOAPTransport.cpp
+++ b/xmltooling/soap/impl/CURLSOAPTransport.cpp
@@ -731,7 +731,6 @@ int xmltooling::verify_callback(X509_STORE_CTX* x509_ctx, void* arg)
     }
 
     if (!success) {
-        log.error("supplied TrustEngine failed to validate SSL/TLS server certificate");
         if (X509_STORE_CTX_get0_cert(x509_ctx)) {
             BIO* b = BIO_new(BIO_s_mem());
             X509_print(b, X509_STORE_CTX_get0_cert(x509_ctx));
@@ -739,10 +738,14 @@ int xmltooling::verify_callback(X509_STORE_CTX* x509_ctx, void* arg)
             BIO_get_mem_ptr(b, &bptr);
             if (bptr && bptr->length > 0) {
                 string s(bptr->data, bptr->length);
-                if (ctx->m_mandatory)
+                if (ctx->m_mandatory) {
+                    log.error("supplied TrustEngine failed to validate SSL/TLS server certificate");
                     log.error(s);
-                else
+                }
+                else {
+                    log.debug("supplied TrustEngine failed to validate SSL/TLS server certificate");
                     log.debug(s);
+                }
             }
             BIO_free(b);
         }

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.


More information about the commits mailing list