commit 372a05a6ac3824483e46f8b1ebce2f5a2fcadb95
author akankshajain18 <cs.akankshajain@gmail.com> Mon Jul 25 10:59:32 2022 +0530
committer GitHub <noreply@github.com> Mon Jul 25 07:29:32 2022 +0200
tree b6b4affa0da2230d73c93be52c17b974fbe12d7b
parent 28b4c77883b8dbdea06e723dc4e5a3eba7393f34

FELIX-6551 | OSGi event blocklisting is not informative (#168)

* FELIX-6551 | OSGi event blocklisting is not informative

* FELIX-6551 | improved logging

* FELIX-6551 | improved logging

* incorporated review changes

eventadmin/impl/src/main/java/org/apache/felix/eventadmin/impl/handler/EventHandlerProxy.java

diff --git a/eventadmin/impl/src/main/java/org/apache/felix/eventadmin/impl/handler/EventHandlerProxy.java b/eventadmin/impl/src/main/java/org/apache/felix/eventadmin/impl/handler/EventHandlerProxy.java
index 30a1d79..a62411f 100644
--- a/eventadmin/impl/src/main/java/org/apache/felix/eventadmin/impl/handler/EventHandlerProxy.java
+++ b/eventadmin/impl/src/main/java/org/apache/felix/eventadmin/impl/handler/EventHandlerProxy.java
@@ -449,10 +449,14 @@
     {
     	if(!this.denied)
     	{
-	        LogWrapper.getLogger().log(
-	                        LogWrapper.LOG_WARNING,
-	                        "Denying event handler from ServiceReference [" + this.reference + " | Bundle("
-	                                        + this.reference.getBundle() + ")] due to timeout!");
+            String output = this.reference + " | Bundle(" + this.reference.getBundle() + ")";
+            if(this.handler != null){
+                output += " | Handler(" + this.handler.getClass().getCanonicalName() + ")";
+            }
+
+            LogWrapper.getLogger().log(
+                    LogWrapper.LOG_ERROR,
+                    String.format("Denying event handler from ServiceReference [%s] due to timeout!", output));
 	        this.denied = true;
 	        // we can free the handler now.
 	        this.release();