Move net.shibboleth.idp.plugin.oidc.op.config from oidc-op-impl into commons
Description
Environment
is related to
Activity
I moved some missing adjacent material, including all the strategy functions and predicates used to access the classes, which will be needed as well. These are in many cases in the saml-api modules for comparison.
The old classes are removed and I’ve refactored the OP code as well as added some missing strategy code to remove most all of the direct config class dependencies from the profile actions.
The OP tests pass for me locally, probably will hit a glitch or two.
I expect we will need to move more code out of the OP packages over time.
One major issue: a couple of the old interfaces were in the OP’s API module such as OIDCProfileConfiguration. There’s really no help for it but to make a policy exception here and remove the old APIs and just make a note of it. I don’t expect that to be a big deal.
Oddly the old AuthenticationContextClassRefPrincipal class was not API, but I wouldn’t be shocked if somebody had a reference to that in a script somewhere, so should also be noted as a change.
I will open a separate ticket for possible renaming or refactoring of the profile classes. Nobody loves the way it’s laid out now but it may be disruptive to change, TBD.
The RP can also benefit from the profile classes in the OP. These should be moved at some point into the commons lib.