Shibboleth FileBackedHTTPMetadataProvider

[Cantor, Scott]

> I was just looking at it.  To special-case it just for *startup* might be hard per
> se, but I think that's not really what you technically want.  Probably want to
> only read the backing file if there is no cached metadata in the resolver.  I
> think that's a purer statement of the requirement.  And that is I think easier
> (although I think I just spotted a minor non-operational bug that would need
> to be fixed first).  I'll open an issue to look at further.

Yes, and I looked into the SP code a little (very little, on an iPad, so not much searching I could do), and from what I can tell, the metadata provider code does have an m_loaded boolean it checks before loading the backup, but oddly a lot of the other reloadable config objects that share that code seem to be ignoring that bit. I really will have to dig, but it seems like I must have realized that flaw, fixed it in one place, but didn't fix it across some of the other pieces.

It's weird enough that I'm going to file a bug on it, but the short of it is, I think the SP is in fact only using the backup metadata file during startup.

-- Scott