Revert to exception-based policy errors, add "secure" flag to policy to track authn.

[shibboleth/cpp-opensaml.git] / saml / binding / SecurityPolicyRule.h

diff --git a/saml/binding/SecurityPolicyRule.h b/saml/binding/SecurityPolicyRule.h
index 68760f7..9a324bb 100644 (file)
--- a/saml/binding/SecurityPolicyRule.h
+++ b/saml/binding/SecurityPolicyRule.h
@@ -45,18 +45,13 @@ namespace opensaml {
         /**
          * Evaluates the rule against the given request and message.
          * 
-         * <p>Exceptions should be reserved for fatal request processing errors;
-         * otherwise rules should return false to indicate they were not applicable
-         * or unsuccessful.
-         * 
          * @param message   the incoming message
          * @param request   the protocol request
          * @param policy    SecurityPolicy to provide various components and track message data
-         * @return  true iff the rule ran successfully, false otherwise
-         * 
-         * @throws BindingException thrown if the request/message is invalid in some way
+         *
+         * @throws BindingException raised if the message/request is not acceptable to the policy rule
          */
-        virtual bool evaluate(
+        virtual void evaluate(
             const xmltooling::XMLObject& message, const GenericRequest* request, SecurityPolicy& policy
             ) const=0;
     };