cmrockwell opened a new pull request #69: URL: https://github.com/apache/sling-whiteboard/pull/69
This PR removed SAML2ConfigService and SAML2ConfigServiceImpl these were not needed, and exposing the information thru the service was not a good idea as mentioned in the ticket. - AuthenticationHandlerSAML2 was changed to an interface - AuthenticationHandlerSAML2Impl was added to org.apache.sling.auth.saml2.impl - That file is busy, so in order not to add more lines of code. AuthenticationHandlerSAML2Impl extends AbstractSamlHandler which set the config properties - Configuration is now a factory with a different base PID, so existing use-cases (probably none exist) will need to redo their OSGI settings ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
