[ 
https://issues.apache.org/jira/browse/SLING-5025?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Konrad Windszus updated SLING-5025:
-----------------------------------
    Attachment: SLING-5025-api.v01.diff
                SLING-5025-resourceresolver.v01.diff

Attached are two patches which
# extend the javadoc for the ResourceResolver to make clear when an ISE can be 
expected
# throw the ISE on all RR methods in case the factory is no longer active

[~cziegeler] Can you have a look?

> Make ResourceResolver methods throw ISE in case the according factory is 
> unregistered
> -------------------------------------------------------------------------------------
>
>                 Key: SLING-5025
>                 URL: https://issues.apache.org/jira/browse/SLING-5025
>             Project: Sling
>          Issue Type: Improvement
>          Components: ResourceResolver
>    Affects Versions: Resource Resolver 1.2.6
>            Reporter: Konrad Windszus
>         Attachments: SLING-5025-api.v01.diff, 
> SLING-5025-resourceresolver.v01.diff
>
>
> In SLING-4360 the {{ResourceResolver.isLive()}} method was extended to check 
> if the {{ResourceResolverFactory}} has been unregistered in the mean time. 
> That check should be done in addition for every method on the resource 
> resolver and should throw an ISE in case the factory is no longer registered.
> See also the discussion at 
> http://www.mail-archive.com/dev@sling.apache.org/msg49364.html.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to