<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<br>
<div class="moz-cite-prefix">On 9/26/12 5:04 PM, Mark O'Quinn wrote:<br>
</div>
<blockquote
cite="mid:DFA6486A-DBF5-46BF-BAB5-7995C1EC4836@gmail.com"
type="cite">
<div>Hi Brent,</div>
<div><br>
</div>
<div>Do you think there's something missing, wrong or that needs
to be improved? Any suggestion is very welcome. </div>
<div><br>
</div>
</blockquote>
<br>
<br>
Aside from what others have already said, this basic approach looks
pretty good to me.<br>
<br>
<br>
<blockquote
cite="mid:DFA6486A-DBF5-46BF-BAB5-7995C1EC4836@gmail.com"
type="cite">
<div>Now, my next step is to add concurrency protection. I saw
some lines (which I'll start study) in the class <b>SAMLMDRelyingPartyConfigurationManager</b>
that I believe are the ones Shibboleth uses:</div>
<div><br>
</div>
<div>
<div><span class="Apple-tab-span" style="white-space:pre"> </span>Lock
readLock = getReadWriteLock().readLock();</div>
<div><span class="Apple-tab-span" style="white-space:pre"> </span>readLock.lock();</div>
</div>
<div><br>
</div>
<div><br>
</div>
</blockquote>
<br>
<br>
I'll respond to this in detail in another message.<br>
<br>
</body>
</html>