fix shouldSkip check (#548)

* fix get version bug * enable cache for get version * add api in all * move enable check to common module * add ark config utils into ark executable jar * get version for non-nested jar * remove unused dep * add dep * inline embed check * fix package npe from cache if empty * update to 2.0.3 * update to 2.0.3-SNAPSHOT * add compatibility for using in inner. * format * add test for finding class from multi libs * revert pom modify * fix should skip Co-authored-by: youji.zzl <[email protected]>
sofastack · Jul 19, 2022 · 29fe6e4 · 29fe6e4
1 parent d1c5a54
commit 29fe6e4
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/.../src/main/java/com/alipay/sofa/ark/support/common/DelegateToMasterBizClassLoaderHook.java b/.../src/main/java/com/alipay/sofa/ark/support/common/DelegateToMasterBizClassLoaderHook.java
@@ -84,8 +84,12 @@ public URL preFindResource(String name, ClassLoaderService classLoaderService, B
 
     @Override
     public URL postFindResource(String name, ClassLoaderService classLoaderService, Biz biz) {
+        if (biz == null || (!biz.isDeclaredMode() && shouldSkip(name))) {
+            return null;
+        }
+
         ClassLoader bizClassLoader = ArkClient.getMasterBiz().getBizClassLoader();
-        if (biz == null || (biz.getBizClassLoader() == bizClassLoader)) {
+        if (biz.getBizClassLoader() == bizClassLoader) {
             return null;
         }
         try {