AspectJ：尝试使用外部aop.xml文件时出现ClassLoading问题

Question

我试图外化aop.xml的配置，所以我从META-INF删除了aop.xml ，并使其在服务器中可供sys admins手动配置。

当我尝试使用外部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 casses。 并且下次它尝试从WebAppClassLoader注册方面时（在加载所有类之后），它工作正常，但我仍然从第一次尝试注册它时记录了异常。

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

当调用以下行时：success = registerAspects（weaver，loader，definitions）;

捕获并记录异常。

    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中提供。

更新：

该功能确实进入了AspectJ 1.7.4，即使它没有在发行说明中明确提及，但在该版本的已解决问题下列出。

Answer 3

我最终做的是将错误消息的LOG Level从ERROR更改为DEBUG，因为我不认为这是一个错误（至少在我的情况下）。 这种情况我启用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));
}