You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Felix Meschberger (JIRA)" <ji...@apache.org> on 2012/11/27 10:35:59 UTC
[jira] [Resolved] (SLING-2299) Consolidate constants and utility
methods
[ https://issues.apache.org/jira/browse/SLING-2299?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Felix Meschberger resolved SLING-2299.
--------------------------------------
Resolution: Fixed
I think this is complete for now. Lets resolve it and have other issues created if something comes up again.
> Consolidate constants and utility methods
> -----------------------------------------
>
> Key: SLING-2299
> URL: https://issues.apache.org/jira/browse/SLING-2299
> Project: Sling
> Issue Type: Improvement
> Components: Authentication
> Affects Versions: Auth Core 1.0.6
> Reporter: Felix Meschberger
> Assignee: Felix Meschberger
> Fix For: Auth Core 1.1.0
>
>
> Currently we have constants spread around class in the Auth Core bundle and we have a number of utility methods in the AbstractAuthenticationHandler. There is even a duplicate isBrowserRequest method in the SlingAuthenticator class (wrongly implemented using the Accept header) and the AuthUtil class.
> We should consolidate the utility methods into the AuthUtil class and the constants into the AuthConstants class.
> This also helps with future evolution: Whenever we add an exported constant or utility method, we have to increase the minor version of the exported package. If this happens on the spi package this would cause all implementations to become incompatible, even though they are not. By having these methods in a separate exported package which containing a single interface not intended to be implemented outside of the auth core bundle, we can increase the minor version of this package without breaking any clients.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira