<html>

  <head>

    <meta content="text/html; charset=windows-1252"

      http-equiv="Content-Type">

  </head>

  <body bgcolor="#FFFFFF" text="#000000">

    <br>

    <br>

    <div class="moz-cite-prefix">On 10/30/15 12:39 AM, Cantor, Scott

      wrote:<br>

    </div>

    <blockquote cite="mid:461CAE8C-216D-4593-89DE-F1D31618A273@osu.edu"

      type="cite">

      <pre wrap="">On 10/29/15, 8:39 PM, "dev on behalf of Brent Putman" <a class="moz-txt-link-rfc2396E" href="mailto:dev-bounces@shibboleth.netonbehalfofputmanb@georgetown.edu"><dev-bounces@shibboleth.net on behalf of putmanb@georgetown.edu></a> wrote:

</pre>

      <blockquote type="cite">

        <pre wrap="">Yeah, that seems off-hand like the obvious way to do it.  And have a new support method on SAMLBindingSupport to eval it, to hide the details.  That could then be usable via a Predicate and/or directly in the XML signing handler, depending on which approach works better.

</pre>

      </blockquote>

      <pre wrap="">Ok, I already coded up most of that anyway. I'll see which layer it makes sense to add the check.</pre>

    </blockquote>

    <br>

    Actually, since this will be a new minor release and we can add API,

    another idea might be to add a Predicate activationCondition to

    AbstractMessageHandler, that is evaled in doPreInvoke(...).   That

    would give all the handlers similar conditional execution capability

    as the actions.<br>

    <br>

    It would have to be a Predicate<MessageContext>.  We could

    have such a predicate for this, I suppose.  But for more general use

    in server-side cases like the IdP where there's a PRC, we could also

    have an adapter allowing use of existing

    Predicate<ProfileRequestContext> impls.  For the latter, we'd

    need to do the thing that I was 99.9% sure we need to do anyway to

    fully support the SOAP client, which is to allow walking via

    MessageContext#getParent() up to the owning PRC (and potentially

    down to the other MessageContext).  I was going to add that

    soon-ish, it's a pretty trivial change.  <br>

    <br>

    I'll look at this more tomorrow, but at the moment this feels like

    the most general and useful solution.<br>

  </body>

</html>