AspectJ:尝试使用外部aop.xml文件时的ClassLoading问题

Question

我试图将aop.xml的配置具体化，因此我从META-INF中删除了aop.xml，并使其在服务器中可供sys管理员手动配置。

当我尝试使用外部aop.xml时

-Dorg.aspectj.weaver.loadtime.configuration="file:D:\Workspace\tomcat7\shared\lib\aop.xml"

我之所以获得java.lang.RuntimeException: Cannot register non aspect: aspectclass....，主要是因为当时AppClassLoader还没有加载aj that。下一次尝试从WebAppClassLoader注册方面时(在加载了所有类之后)，它可以正常工作，但我仍然可以从第一次尝试注册它时得到记录的异常。

异常被捕获并记录在ClassLoaderWeavingAdaptor.java第307行。

当调用以下行时: success =registerAspects(织布器、加载器、定义)；

异常将被捕获并记录。

    try {
        registerOptions(weaver, loader, definitions);
        registerAspectExclude(weaver, loader, definitions);
        registerAspectInclude(weaver, loader, definitions);
        success = registerAspects(weaver, loader, definitions);
        registerIncludeExclude(weaver, loader, definitions);
        registerDump(weaver, loader, definitions);
    } catch (Exception ex) {
        trace.error("register definition failed", ex);
        success = false;
        warn("register definition failed", (ex instanceof AbortException) ? null : ex);
    }

异常在BcelWeaver.java中的以下行中异常抛出

if (type.isAspect()) {
      ......
} else {
        // FIXME AV - better warning upon no such aspect from aop.xml
        RuntimeException ex = new RuntimeException("Cannot register non aspect: " + type.getName() + " , " + aspectName);
        if (trace.isTraceEnabled()) {
            trace.exit("addLibraryAspect", ex);
        }
        throw ex;
    }

当方面尚未加载时，如何防止classLoader将错误记录到控制台。我想对记录源文件异常的行进行注释，并重新构建aspectjweaver jar文件，但在不修改aspectj源代码的情况下寻找更好的解决方案。

Answer 1

我不确定有什么容易的办法解决你的问题。正如我说的，我以前没有和AspectJ合作过，但我相信这是织工的错误行为。

问题描述:在引导期间，代理尝试将编织应用到其他方面，不仅适用于WebAppClassLoader，还应用于整个类加载器链(每个类加载器一次)，即：sun.misc.Launcher$AppClassLoader、sun.misc.Launcher$ExtClassLoader、org.apache.catalina.loader.StandardClassLoader (tomcat的类加载器)。当您使用META-INF/aop.xml方法时，由于“配置文件不可用”(如果启用verbose模式，您可以在控制台中看到这些消息)，它将禁用上述类加载器的编织。使用文件配置方法时，链中的所有类加载器都可以使用配置。由于它确实找到了一个配置文件，代理解析了定义，所以它找不到方面的类并显示错误。

奇怪的是，正如配置文档中所描述的，如果您使用WeavingURLClassLoader方法来加载时间编织，“.它还允许用户通过类加载器显式地限制可以编织哪些类”。所以这实际上是一个特性(!)类加载器方法可能有，但代理方法没有。(不幸的是，我无法使用这种方法)

好消息(和坏消息)：好消息是，您可以轻松地创建您自己的代理，它将忽略前面提到的类加载器的编织。坏消息是，限制每个类加载器的编织是不够的，因为如果您在同一台服务器中有其他应用程序，Tomcat仍然会使用WebAppClassLoader加载它们，因此仍然会得到这些应用程序的错误消息。(在这种情况下，您也可以扩展下面的类来筛选包/类)。

下面您可以为修改的代理找到两个类。要使用它们，您需要执行以下操作：

• 将aspectjweaver.jar放到一个文件夹中
• 在org/aspectj/weaver/loadtime下创建一个新文件夹filter以匹配包名，并在编译后将两个新类放在那里。
• 编辑META-INF/MANIFEST.MF文件并更改行 Premain-Class: org.aspectj.weaver.loadtime.Agent to Premain-Class: org.aspectj.weaver.loadtime.filter.FilterAgent
• 你的新探员已经准备好了。
• 在启动JVM时，您现在可以传递一个带有逗号分隔的类加载器列表的新系统属性，您希望忽略它，即-Dorg.aspectj.weaver.loadtime.filter=sun.misc.Launcher$AppClassLoader,sun.misc.Launcher$ExtClassLoader,org.apache.catalina.loader.StandardClassLoader (我已将CATALINA_OPTS设置为这样做)。

这些类是原始代理类Agent和ClassPreProcessorAgentAdapter的修改副本。我添加的唯一代码是解析上述系统属性(如果存在)的部分，并忽略对我们不感兴趣的类加载器的调用。

我希望这对你有帮助。

package org.aspectj.weaver.loadtime.filter;

import java.lang.instrument.ClassFileTransformer;
import java.lang.instrument.Instrumentation;

public class FilterAgent {

    private static Instrumentation s_instrumentation;

    // Use our own version of ClassFileTransformer that would filter out selected classloaders 
    private static ClassFileTransformer s_transformer = new ClassPreprocessorFilteredAdapter();

    /**
     * JSR-163 preMain Agent entry method
     *
     * @param options
     * @param instrumentation
     */
    public static void premain(String options, Instrumentation instrumentation) {
        /* Handle duplicate agents */
        if (s_instrumentation != null) {
            return;
        }
        s_instrumentation = instrumentation;
        s_instrumentation.addTransformer(s_transformer);
    }

    public static Instrumentation getInstrumentation() {
        if (s_instrumentation == null) {
            throw new UnsupportedOperationException("Java 5 was not started with preMain -javaagent for AspectJ");
        }
        return s_instrumentation;
    }
}
//-----------------------------------------------------------------------------------
package org.aspectj.weaver.loadtime.filter;

import java.lang.instrument.ClassFileTransformer;
import java.lang.instrument.IllegalClassFormatException;
import java.security.ProtectionDomain;
import java.util.HashMap;
import java.util.Map;

import org.aspectj.weaver.loadtime.Aj;
import org.aspectj.weaver.loadtime.ClassPreProcessor;


public class ClassPreprocessorFilteredAdapter implements ClassFileTransformer {

    /**
     * Concrete preprocessor.
     */
    private static ClassPreProcessor s_preProcessor;

    private static Map<String, String> ignoredClassloaderNames = new HashMap<String, String>();

    static {
        try {
            s_preProcessor = new Aj();
            s_preProcessor.initialize();


            String ignoredLoaders = System.getProperty("org.aspectj.weaver.loadtime.filter", "");
            if (ignoredLoaders.length() > 0) {
                String[] loaders = ignoredLoaders.split(",");

                for (String s : loaders) {
                    s = s.trim();
                    ignoredClassloaderNames.put(s, s);
                    System.out.println("---> Will filtered out classloader: " + s);
                }
            }

        } catch (Exception e) {
            throw new ExceptionInInitializerError("could not initialize JSR163 preprocessor due to: " + e.toString());
        }
    }

    /**
     * Invokes the weaver to modify some set of input bytes.
     * 
     * @param loader the defining class loader
     * @param className the name of class being loaded
     * @param classBeingRedefined is set when hotswap is being attempted
     * @param protectionDomain the protection domain for the class being loaded
     * @param bytes the incoming bytes (before weaving)
     * @return the woven bytes
     */
    @Override
    public byte[] transform(ClassLoader loader, String className, Class<?> classBeingRedefined, ProtectionDomain protectionDomain, byte[] bytes) throws IllegalClassFormatException {
        if (classBeingRedefined != null) {
            System.err.println("INFO: (Enh120375):  AspectJ attempting reweave of '" + className + "'");
        }

        String loaderName = loader.getClass().getName();
        if (shouldIgnoreClassLoader(loaderName)) {
            return bytes;
        }
        return s_preProcessor.preProcess(className, bytes, loader, protectionDomain);
    }

    private boolean shouldIgnoreClassLoader(String loaderName) {
        boolean result = false;
        String ignoredLoader = ignoredClassloaderNames.get(loaderName);
        if (ignoredLoader != null) {
            result = true;    // if the loader name exists in the map we will ignore weaving
        }
        return result;
    }
}

Answer 2

如果您需要将类加载器排除在使用代理方法编织的范围之外，那么就有一个开发人员构建提供了一个新的命令行开关-Daj.weaving.loadersToSkip来实现这个功能。这个主题正在AspectJ用户邮件列表的线程上讨论。这个特性可能会被写入AspectJ 1.7.4中，但在1.7.3中还没有实现。

更新：

尽管功能在发布说明中没有明确提到，但是它被列在了已解决的问题下面，但它确实被写入了发布说明 1.7.4中。

Answer 3

我最后所做的是将错误消息的日志级别从error更改为DEBUG，因为我不认为这是错误(至少在我的情况下)。在启用调试级别时，我仍然可以看到错误。因此，我修改了下面的源文件并重新构建了aspectjweaver-1.7.1.jar

try {
    registerOptions(weaver, loader, definitions);
    registerAspectExclude(weaver, loader, definitions);
    registerAspectInclude(weaver, loader, definitions);
    success = registerAspects(weaver, loader, definitions);
    registerIncludeExclude(weaver, loader, definitions);
    registerDump(weaver, loader, definitions);
} catch (Exception ex) {
    //(CHANGE 1) trace.error("register definition failed", ex);
    trace.debug( "register definition failed" + ex.getMessage());
    success = false;
    // (CHANGE 2) warn("register definition failed", (ex instanceof AbortException) ? null : ex);
    debug("register definition failed" + ((ex instanceof AbortException) ? null : ex));
}