<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/29/14 11:00 AM, Cantor, Scott
wrote:<br>
</div>
<blockquote cite="mid:CFACC28B.DAB3%25cantor.2@osu.edu" type="cite">
<blockquote type="cite">
<pre wrap="">Question: do we think I am missing any other significant uses of these
classes in the wild, or in our own code?
</pre>
</blockquote>
<pre wrap="">
Nothing I've done. I was assuming that the custom schema parsing was using
them, but it looks like they call KeySupport-type functions directly.</pre>
</blockquote>
<br>
<br>
I don't know of anything either. I always assumed Chad did them
primarily to support the MDA and/or xmlsectool. Other than that, I
just know of the stuff in the v3 IdP. <br>
<br>
<blockquote cite="mid:CFACC28B.DAB3%25cantor.2@osu.edu" type="cite">
<pre wrap="">
</pre>
<blockquote type="cite">
<pre wrap="">I think File was the wrong starting point for these classes, so my
preference would be to remove the existing methods with the File
argument, establish new methods taking Resource instead, and adjust
configurations everywhere as required.
</pre>
</blockquote>
<pre wrap="">
Yes, I would think so.</pre>
</blockquote>
<br>
I agree, Resource is more flexible and makes more sense there.<br>
<br>
<br>
<blockquote cite="mid:CFACC28B.DAB3%25cantor.2@osu.edu" type="cite">
<pre wrap="">
</pre>
<blockquote type="cite">
<pre wrap="">Question: any rationale for
setCertificateResource/setPrivateKeyResource/etc. vs. setResource?
</pre>
</blockquote>
<pre wrap="">
Depends where the methods live I guess. If you have a class that has to
take multiple separate resources, we'd need to clarify which was which.</pre>
</blockquote>
<br>
<br>
Agreed. Doesn't look like any of the current ones take more than
one File/resource type thingy. I personally don't have a strong
preference.<br>
<br>
<br>
<blockquote cite="mid:CFACC28B.DAB3%25cantor.2@osu.edu" type="cite">
<pre wrap="">
</pre>
<blockquote type="cite">
<pre wrap="">Bottom line: I have reasons to want to just get this cleared out now, and
apart from those I also think this deserves to be fixed for the IdP
alpha. Unless there's a good reason against doing this, I'd like to just
get it done.
</pre>
</blockquote>
<pre wrap="">
Please.
</pre>
</blockquote>
<br>
I second the motion!<br>
<br>
<br>
<br>
<br>
</body>
</html>