[cpp-sp] branch master updated: SSPCPP-831 - Review PropertySet options affected by namespace change

Scott Cantor cantor.2 at osu.edu
Thu Jul 26 12:02:02 EDT 2018


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=6c6e978dc73c232eaaf90fe9573ba13e0b679402

The following commit(s) were added to refs/heads/master by this push:
       new  6c6e978   SSPCPP-831 - Review PropertySet options affected by namespace change
6c6e978 is described below

commit 6c6e978dc73c232eaaf90fe9573ba13e0b679402
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Thu Jul 26 12:01:11 2018 -0400

    SSPCPP-831 - Review PropertySet options affected by namespace change
    
    https://issues.shibboleth.net/jira/browse/SSPCPP-831
---
 shibsp/handler/impl/AbstractHandler.cpp | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/shibsp/handler/impl/AbstractHandler.cpp b/shibsp/handler/impl/AbstractHandler.cpp
index 75d3e83..e808957 100644
--- a/shibsp/handler/impl/AbstractHandler.cpp
+++ b/shibsp/handler/impl/AbstractHandler.cpp
@@ -508,8 +508,9 @@ long AbstractHandler::sendMessage(
 {
     const EntityDescriptor* entity = role ? dynamic_cast<const EntityDescriptor*>(role->getParent()) : nullptr;
     const PropertySet* relyingParty = application.getRelyingParty(entity);
-    pair<bool,const char*> flag = getString("signing",
-        !getElement() || XMLString::equals(getElement()->getNamespaceURI(), shibspconstants::SHIBSPCONFIG_NS) ? nullptr : shibspconstants::ASCII_SHIBSPCONFIG_NS);
+    pair<bool, const char*> flag = getString("signing");
+    if (!flag.first)
+        flag = getString("signing", shibspconstants::ASCII_SHIBSPCONFIG_NS);
     if (!flag.first)
         flag = relyingParty->getString("signing");
     if (SPConfig::shouldSignOrEncrypt(flag.first ? flag.second : defaultSigningProperty, destination, encoder.isUserAgentPresent())) {

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


More information about the commits mailing list