<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<br>
<div class="moz-cite-prefix">On 5/30/14 8:46 PM, Cantor, Scott
wrote:<br>
</div>
<blockquote cite="mid:CFAE9DCC.4ECB8%25cantor.2@osu.edu" type="cite">
<pre wrap="">
I remember discussing something else, I didn't go looking for it. I was
more concerned about configuring this than implementing it, I just needed
something to test with.
The filter implementation was done a while ago, I was just adding some
error checking.</pre>
</blockquote>
<br>
<br>
That's fine, I just wanted to make sure we weren't forgetting about
this, in general. In the past Chad, and to a certain extent Ian,
were saying EntitiesDescriptors == Bad.<br>
<br>
<blockquote cite="mid:CFAE9DCC.4ECB8%25cantor.2@osu.edu" type="cite">
<pre wrap="">
</pre>
<pre wrap="">
I don't think either Rod nor myself are familiar with that code so it's
not currently part of the configuration logic. We would presumably have to
bake this into the default metadata configuration parsing code, because if
the group policy implementations relied on it, they'd have to be installed
without the user actually putting them in, since they're not in the V2
configuration now.</pre>
</blockquote>
<br>
Oh right, for the legacy config. Yeah, it's very easy, just a
single filter with these 1 or 2 node processor plugins. For the
Spring native config, it's really just a bit of additional wiring,
nothing complicated about it.<br>
<br>
<br>
<blockquote cite="mid:CFAE9DCC.4ECB8%25cantor.2@osu.edu" type="cite">
<pre wrap="">
</pre>
<blockquote type="cite">
<pre wrap="">Walking the tree for the EntitiesDescriptors as below does currently
work, because the metadata resolvers don't currently destroy or mutate
the tree structure. But with our new EntityDescriptor-centric focus,
that's not guaranteed to always be the case - unless we decide that
preserving it is a requirement, but that would be moving in the opposite
direction from what I thought we wanted.
</pre>
</blockquote>
<pre wrap="">
Preserving the functionality is a requirement, but if we're going to do it
a different way, we probably need an issue to track that.</pre>
</blockquote>
<br>
I don't have any plans to destroy the tree, I'll probably just leave
it the way it is, presumably doesn't hurt anything. I just thought
it best to reserve the right to change it, if we want. <br>
<br>
The idea was that the only thing that should ever look at
EntitiesDescriptors was the metadata resolver(s) themselves. Once
the metadata is processed, callers don't know or care about them
anymore.<br>
<br>
<br>
<blockquote cite="mid:CFAE9DCC.4ECB8%25cantor.2@osu.edu" type="cite">
<pre wrap="">
Is the code for these new filters complete?</pre>
</blockquote>
<br>
Yes, as far as I know. Barring any undiscovered bugs, etc.<br>
<br>
</body>
</html>