X-Git-Url: http://www.project-moonshot.org/gitweb/?a=blobdiff_plain;f=saml%2Fsaml1%2Fbinding%2Fimpl%2FSAML1SOAPDecoder.cpp;h=78888241a757c10f010818fce12863843fe5a783;hb=c2f3cb788f744ebb8d89b22e0676b68737abfa9f;hp=d759507b2ae628df05d8b02a93b6e9cf421139ae;hpb=f9aa9375e0908e119af283278d00468a9b2249f5;p=shibboleth%2Fopensaml2.git diff --git a/saml/saml1/binding/impl/SAML1SOAPDecoder.cpp b/saml/saml1/binding/impl/SAML1SOAPDecoder.cpp index d759507..7888824 100644 --- a/saml/saml1/binding/impl/SAML1SOAPDecoder.cpp +++ b/saml/saml1/binding/impl/SAML1SOAPDecoder.cpp @@ -42,9 +42,13 @@ namespace opensaml { class SAML_DLLLOCAL SAML1SOAPDecoder : public MessageDecoder { public: - SAML1SOAPDecoder(const DOMElement* e) {} + SAML1SOAPDecoder() {} virtual ~SAML1SOAPDecoder() {} - + + bool isUserAgentPresent() const { + return false; + } + xmltooling::XMLObject* decode( std::string& relayState, const GenericRequest& genericRequest, @@ -52,9 +56,9 @@ namespace opensaml { ) const; }; - MessageDecoder* SAML_DLLLOCAL SAML1SOAPDecoderFactory(const DOMElement* const & e) + MessageDecoder* SAML_DLLLOCAL SAML1SOAPDecoderFactory(const pair& p) { - return new SAML1SOAPDecoder(e); + return new SAML1SOAPDecoder(); } }; }; @@ -74,14 +78,12 @@ XMLObject* SAML1SOAPDecoder::decode( string s = genericRequest.getContentType(); if (s.find("text/xml") == string::npos) { log.warn("ignoring incorrect content type (%s)", s.c_str() ? s.c_str() : "none"); - return NULL; + throw BindingException("Invalid content type for SOAP message."); } const char* data = genericRequest.getRequestBody(); - if (!data) { - log.warn("empty request body"); - return NULL; - } + if (!data) + throw BindingException("SOAP message had an empty request body."); istringstream is(data); // Parse and bind the document into an XMLObject.