(tomee) branch tomee-9.x updated: feat: improve CdiScanner logging and dependencies issues only when activated which is now true by default.

[jlmonteiro]

This is an automated email from the ASF dual-hosted git repository.

jlmonteiro pushed a commit to branch tomee-9.x
in repository https://gitbox.apache.org/repos/asf/tomee.git


The following commit(s) were added to refs/heads/tomee-9.x by this push:
     new b0d2b3d059 feat: improve CdiScanner logging and dependencies issues 
only when activated which is now true by default.
b0d2b3d059 is described below

commit b0d2b3d05983045df775a028c8fe7df4427745c1
Author: Jean-Louis Monteiro <jlmonte...@tomitribe.com>
AuthorDate: Tue May 6 10:50:18 2025 +0200

    feat: improve CdiScanner logging and dependencies issues only when 
activated which is now true by default.
    
    (cherry picked from commit 1e3ec81f6d153a7cd036cbb6bbf2fe10be6d8449)
---
 .../src/main/java/org/apache/openejb/cdi/CdiScanner.java         | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git 
a/container/openejb-core/src/main/java/org/apache/openejb/cdi/CdiScanner.java 
b/container/openejb-core/src/main/java/org/apache/openejb/cdi/CdiScanner.java
index b05d7b2cce..50b3b59bbc 100644
--- 
a/container/openejb-core/src/main/java/org/apache/openejb/cdi/CdiScanner.java
+++ 
b/container/openejb-core/src/main/java/org/apache/openejb/cdi/CdiScanner.java
@@ -90,7 +90,7 @@ public class CdiScanner implements BdaScannerService {
 
 
     public CdiScanner() {
-        logDebug = 
"true".equals(SystemInstance.get().getProperty("openejb.cdi.noclassdeffound.log",
 "false"));
+        logDebug = 
"true".equals(SystemInstance.get().getProperty("openejb.cdi.noclassdeffound.log",
 "true"));
 
         tomeeBeanArchiveInformation = new 
DefaultBeanArchiveInformation("tomee");
         
tomeeBeanArchiveInformation.setBeanDiscoveryMode(BeanArchiveService.BeanDiscoveryMode.ALL);
@@ -387,16 +387,21 @@ public class CdiScanner implements BdaScannerService {
             tryToMakeItFail(loadClass);
             return loadClass;
         } catch (final ClassNotFoundException e) {
+            Logger.getInstance(LogCategory.OPENEJB_CDI, 
CdiScanner.class).debug("Can not load {0}.", e, className);
             return null;
         } catch (final NoClassDefFoundError e) {
             if (logDebug) {
-                Logger.getInstance(LogCategory.OPENEJB_CDI, 
CdiScanner.class).warning(className + " -> " + e);
+                Logger.getInstance(LogCategory.OPENEJB_CDI, 
CdiScanner.class).warning("Can not load {0} dependencies.", e, className);
             }
+            Logger.getInstance(LogCategory.OPENEJB_CDI, 
CdiScanner.class).debug("Can not load {0} dependencies.", e, className);
             return null;
         }
     }
 
     private void tryToMakeItFail(final Class<?> loadClass) { // we try to 
avoid later NoClassDefFoundError
+        if (!logDebug) {
+            return;
+        }
         loadClass.getDeclaredFields();
         loadClass.getDeclaredMethods();
     }