<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Mon, Aug 1, 2016 at 11:16 AM Cantor, Scott <<a href="mailto:cantor.2@osu.edu">cantor.2@osu.edu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Why do you think incorrectly? My expiring password check is in that hook (it's the one I checked into trunk).<br></blockquote><div><br></div><div>I hadn't seen that, thanks for mentioning it. Careful review reminded me of the<span style="line-height:1.5"> ProfileConfiguration knobs (</span>inboundInterceptorFlows, <span style="line-height:1.5">postAuthenticationFlows, </span>outboundInterceptorFlows) <span style="line-height:1.5">that control when the intercept flows fire. We are indeed using those to ensure that our custom intercept flows fire post-authentication, but I had forgot and missed it in my review. Also, I see it's fairly clearly documented on </span><a href="https://wiki.shibboleth.net/confluence/display/IDP30/ProfileInterceptConfiguration#ProfileInterceptConfiguration-EnablingIntercepts">https://wiki.shibboleth.net/confluence/display/IDP30/ProfileInterceptConfiguration#ProfileInterceptConfiguration-EnablingIntercepts</a>. So I misspoke about our misuse of intercept flows. Working as intended, sorry.</div><div><span style="line-height:1.5"><br></span></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
> 3. Authentication method check based on 2-factor completion<br>
You shouldn't need that one at all, it would mean the authentication flow(s) themselves need to be fixed.</blockquote><div><br></div><div>The authentication method is not static but a function of the data in the AttributeContext. I don't know whether that's right or wrong, but those are the requirements. I don't see any other way to accomplish it than via an intercept flow that fiddles with the authenticated subject under certain conditions.</div><div> </div><div>Thanks,</div><div>M</div><div><br></div></div></div>