Question

我正在尝试外化aop.xml的配置，因此我从aop.xml中删除了META-INF并使其在服务器中可供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 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模式，则可以在控制台中看到这些消息）。使用文件配置方法时，配置可用于链中的所有类加载器。由于它确实找到了配置文件，因此代理会解析定义，但它找不到方面的类并显示错误。

奇怪的是，如configuration documentation中所述，如果您使用WeavingURLClassLoader方法进行加载时间编织，“...它还允许用户按类显式限制装载机哪些类可编织“。所以这实际上是类加载器方法可以具有的功能（！），但代理方法却没有。（不幸的是我无法使用这种方法）

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

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

将aspectjweaver.jar解压缩到文件夹
在org/aspectj/weaver/loadtime下创建一个新文件夹filter以匹配包名称，并在编译后将两个新类放在那里。
编辑META-INF/MANIFEST.MF文件并更改行

Premain-Class: org.aspectj.weaver.loadtime.Agent到
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

如果您需要使用代理方法排除类加载器的功能，可以使用developer build提供新的命令行开关-Daj.weaving.loadersToSkip来执行此操作。该主题正在thread of the AspectJ users mailing list进行讨论。该功能可能会进入AspectJ 1.7.4，但尚未在1.7.3中提供。

<强>更新

feature确实进入了AspectJ 1.7.4，即使它未在release notes中明确提及，但在resolved issues下列为该版本。

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));
}

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

3 个答案: