[JCRVLT-116] InstallHookProcessor.Hook should not only use TCCL as fallback

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.1.26
• Fix Version/s: 3.1.28
• Assignee: Unassigned
• Reporter: Konrad Windszus
• Created: 18/Apr/16 13:28
• Updated: 04/Dec/17 18:41
• Resolved: 18/Aug/16 05:57

Description

Currently the InstallHookProcessor.Hook only uses the thread context class loader as fallback to load the hook classes (https://github.com/apache/jackrabbit-filevault/blob/64caa29325452c1adf968ae05e2308b37f3600cc/vault-core/src/main/java/org/apache/jackrabbit/vault/packaging/impl/InstallHookProcessorImpl.java#L206). This almost never works in an OSGi context.

Instead I would propose to try the Bundle's Class Loader and add `DynamicImport-Package: *` to the OSGi manifest.
That of course would not work in non-OSGi contexts. Therefore it should just be implemented as another fallback option (in addition to TCCL).

Issue Links

• is related to: JCRVLT-248 Support install hook bundles