fix inner guice class usage in always executable code (OSGi issue) (f…

…ixes #187)
xvik · Dec 19, 2021 · 5786ae9 · 5786ae9
1 parent f3084f6
commit 5786ae9
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,5 @@
+* Fix inner guice class usage in always executable code (#187, OSGi issue)
+
 ### 5.4.0 (2021-10-21)
 * Use direct dependency versions in pom to simplify resolution (dependencyManagement section remains but for usage as BOM only)
 * Fix lambda modules support (modules declared with lambda expression) (#160)

diff --git a/src/main/java/ru/vyarus/dropwizard/guice/module/context/stat/GuiceStatsTracker.java b/src/main/java/ru/vyarus/dropwizard/guice/module/context/stat/GuiceStatsTracker.java
@@ -55,7 +55,9 @@ public List<String> getMessages() {
     }
 
     private Logger getLogger() {
-        return Logger.getLogger(com.google.inject.internal.util.ContinuousStopwatch.class.getName());
+        // string used instead of safer class reference due to OSGI issue (#187)
+        // (internal package reference not possible - cause class not found)
+        return Logger.getLogger("com.google.inject.internal.util.ContinuousStopwatch");
     }
 
     /**