[cpp-sp] branch master updated: Fix some slightly confusing log messages.

Scott Cantor cantor.2 at osu.edu
Fri Oct 4 12:36:28 EDT 2019


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

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

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

The following commit(s) were added to refs/heads/master by this push:
       new  f788fab   Fix some slightly confusing log messages.
f788fab is described below

commit f788fabb32c0c0884522e61ac692973a439c7448
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Fri Oct 4 12:36:20 2019 -0400

    Fix some slightly confusing log messages.
---
 plugins/GSSAPIAttributeExtractor.cpp                     | 2 +-
 shibsp/attribute/resolver/impl/XMLAttributeExtractor.cpp | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/plugins/GSSAPIAttributeExtractor.cpp b/plugins/GSSAPIAttributeExtractor.cpp
index fadfb25..096d4ad 100644
--- a/plugins/GSSAPIAttributeExtractor.cpp
+++ b/plugins/GSSAPIAttributeExtractor.cpp
@@ -250,7 +250,7 @@ void GSSAPIExtractorImpl::extractAttributes(
     string attrname(reinterpret_cast<char*>(namingAttribute->value), namingAttribute->length);
     map<string,Rule>::const_iterator rule = m_attrMap.find(attrname);
     if (rule == m_attrMap.end()) {
-        m_log.info("skipping unmapped GSS-API attribute: %s", attrname.c_str());
+        m_log.info("skipping GSS-API attribute: %s", attrname.c_str());
         return;
     }
 
diff --git a/shibsp/attribute/resolver/impl/XMLAttributeExtractor.cpp b/shibsp/attribute/resolver/impl/XMLAttributeExtractor.cpp
index b3366a7..1111b3f 100644
--- a/shibsp/attribute/resolver/impl/XMLAttributeExtractor.cpp
+++ b/shibsp/attribute/resolver/impl/XMLAttributeExtractor.cpp
@@ -444,7 +444,7 @@ void XMLExtractorImpl::extractAttributes(
     }
     else if (m_log.isDebugEnabled()) {
         auto_ptr_char temp(format);
-        m_log.debug("skipping unmapped NameIdentifier with format (%s)", temp.get());
+        m_log.debug("skipping NameIdentifier with format (%s)", temp.get());
     }
 }
 
@@ -469,7 +469,7 @@ void XMLExtractorImpl::extractAttributes(
     }
     else if (m_log.isDebugEnabled()) {
         auto_ptr_char temp(format);
-        m_log.debug("skipping unmapped NameID with format (%s)", temp.get());
+        m_log.debug("skipping NameID with format (%s)", temp.get());
     }
 }
 
@@ -499,7 +499,7 @@ void XMLExtractorImpl::extractAttributes(
     else if (m_log.isInfoEnabled()) {
         auto_ptr_char temp1(name);
         auto_ptr_char temp2(format);
-        m_log.info("skipping unmapped SAML 1.x Attribute with Name: %s%s%s", temp1.get(), *temp2.get() ? ", Namespace:" : "", temp2.get());
+        m_log.info("skipping SAML 1.x Attribute with Name: %s%s%s", temp1.get(), *temp2.get() ? ", Namespace:" : "", temp2.get());
     }
 }
 
@@ -544,7 +544,7 @@ void XMLExtractorImpl::extractAttributes(
     if (m_log.isInfoEnabled()) {
         auto_ptr_char temp1(name);
         auto_ptr_char temp2(format);
-        m_log.info("skipping unmapped SAML 2.0 Attribute with Name: %s%s%s", temp1.get(), *temp2.get() ? ", Format:" : "", temp2.get());
+        m_log.info("skipping SAML 2.0 Attribute with Name: %s%s%s", temp1.get(), *temp2.get() ? ", Format:" : "", temp2.get());
     }
 }
 

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


More information about the commits mailing list