Add metrics for installed plugins and modules

Added the two new metrics beans to https://shibboleth.atlassian.net/wiki/spaces/IDP5/pages/3199510278/MetricsConfiguration .

If we chose to document the two properties which control the HTTP client and secuirty parameters then that is as good a place as any. For the record the properties are currently called idp.updateCheck.httpClient and idp.updateCheck.httpSecurityParameters (and of course they are shared with the class which talks queries the IdP update state at container start.

I’ll resolve this case for now, but not close it. It feels like there will be more work to be done when we start Beta testing

I renamed installedcomponent to installation just for brevity, but I’ll be tweaking names up to the final day so whatever it is for now is fine. I verified that adding this to logback.xml bypasses any of that code, so it’s “safe” for us to ship.

I am inclined to leave it on by default and make sure that the User Agents we see in the log are useful to us. If we’re going to phone home, let’s be sure it’s helpful. We’ll obviously need to document that in the notes.

(after which back to me for release notes and doco)

Documentation pending, but code committed

I have no investment in the names either of the metrics or of the classes so if they are wrong people shgould say so, or just change the code.

Note that with all the above turned on, and with only shibboleth plugins installed collecting the metric set will only go off site twice 9once for the IdP update info and once for all the plugins. The cache of plugin information is cached (including a negative cache).

I’ll ask @Scott Cantor to review and to add any filters he feels might be needed.

I generally try and use classes because that’s more flexible (it will use toString if there are no serializing rules, but we can plug in rules into Jackson where desired to control it better. So using classes defers the decision longer.

Nulls should be fine.

“Not much” generally, but if there are actionable warnings that’s probably fine.