Shibboleth Developer's Meeting, May 24, 2013

...

As documented in Web Flow, when a flow reaches an end-state, if a view-state is not rendered the default behavior is to redirect to beginning of flow. Need to mark response as completed at end of flow, RequestContext.getExternalContext().recordResponseComplete().

Daniel

Ian

 Task dies if ThreadExecutor throws an exception ? In vt-ldap, if a periodic validator is configured and the host is unavailable and an exception is thrown, the thread will disappear.

Scott : Should v2 depend on ldaptive ?

Daniel : Would take some work.

Make a note of issue, and defer making changes until we see further need to.

Ian

10:50

Marvin

Rod

• Attribute Filtering.  After discussion with Tom, completed refactoring (unfortunately some may need unrefactoring)
• Started on the the rest of the "Policy" type predicates.
• Design note for AttributeInMetadata and AttributeInRequest under  construction.

...

Refactoring idp-attribute-filter-api, looks like a return to v2, with some differences. Not sure why v3 diverged originally, let us avoid design regressions.

Be deliberate when changing API artifacts. Special svn notifications ?

Continue to refactor "MatchFunctor" to "Matcher" and its evaluatePolicyRule(AttributeFilterContext) to matches(AttributeFilterContext).

Q : "What does a Matcher do?"
A : "A Matcher matches (an attribute filter context) and it gets matching values (for an attribute in the context)".

As mentioned in Javadoc, a base AbstractMatcher class could require concrete implementations to provide just getMatchingValues(), since a default matches() implementation could iterate over the Attributes in the AttributeFilterContext and return true the first time getMatchingValues() returns a non-empty set. Not optimal, of course.

So, MatchFunctor might exist in the schema only.

As mentioned on a previous dev call, for consistency, I thing "filtering" should be refactored to "filter" in Java class and package names, and the AttributeFilteringEngine should become AttributeFilter. To match the AttributeResolver maybe AttributeFilterer is more appropriate, but no.

Readlock on AttributeFilterService.filterAttributes() ?

Topics

Decisions

Coding convention : getLdapUrl or getLDAPURL ?

...