felix-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Richard S. Hall (JIRA)" <j...@apache.org>
Subject [jira] Closed: (FELIX-2670) [Framework] Implicit boot delegation doesn't delegate for external code in all cases
Date Mon, 25 Oct 2010 20:24:20 GMT

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

Richard S. Hall closed FELIX-2670.
----------------------------------

    Resolution: Fixed
      Assignee: Richard S. Hall

I've applied a patch for this and have created several test cases for it that all pass. I
also tested it against the scenario in FELIX-2653. Everything looks good, but it is possible
that I'm missing something. We'll just address any regressions as they are discovered.

> [Framework] Implicit boot delegation doesn't delegate for external code in all cases
> ------------------------------------------------------------------------------------
>
>                 Key: FELIX-2670
>                 URL: https://issues.apache.org/jira/browse/FELIX-2670
>             Project: Felix
>          Issue Type: Bug
>          Components: Framework
>    Affects Versions: framework-3.0.5
>            Reporter: Richard S. Hall
>            Assignee: Richard S. Hall
>            Priority: Minor
>             Fix For: framework-3.2.0
>
>         Attachments: FELIX-2670.txt
>
>
> When determining if a class is not from a bundle in ModuleImpl.isClassNotLoadedFromBundle()
(terrible method name, by the way), it attempts to ignore all framework classes by checking
if the target class' class loader is the same as the framework's class loader. This check
is too broad since the framework's class loader is typically the app class loader, which means
that any other code on the class path will not trigger implicit boot delegation.
> You could argue that implicit boot delegation was intended for JRE code and this is ok.
However, just by changing the framework launcher to use a different class loader for the framework,
you would get implicit boot delegation for code on the class path, so it seems odd that we
trigger it under one situation but not another.
> Perhaps the simple solution is to not perform a class loader check, but to compare the
package namespace (i.e., use org.apache.felix.framework to determine framework classes).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message