log a message if a plugin was loaded but not found in auth stack

Vladimir Kotal · Vladimir Kotal · commit cde1ce29c38e · 2017-11-15T17:02:40.000+01:00
diff --git a/src/org/opensolaris/opengrok/authorization/AuthorizationFramework.java b/src/org/opensolaris/opengrok/authorization/AuthorizationFramework.java
@@ -460,7 +460,10 @@ private void loadClasses(AuthorizationStack stack, List<File> classfiles, List<F
             }
             // Load the class in memory and try to find a configured space for this class.
             if ((pf = handleLoadClass(classname)) != null) {
-                stack.setPlugin(pf);
+                if (!stack.setPlugin(pf)) {
+                    LOGGER.log(Level.INFO, "plugin {0} is not configured in the stack",
+                            classname);
+                }
             }
         }
 
@@ -475,7 +478,10 @@ private void loadClasses(AuthorizationStack stack, List<File> classfiles, List<F
                     }
                     // Load the class in memory and try to find a configured space for this class.
                     if ((pf = handleLoadClass(classname)) != null) {
-                        stack.setPlugin(pf);
+                        if (!stack.setPlugin(pf)) {
+                            LOGGER.log(Level.INFO, "plugin {0} is not configured in the stack",
+                                    classname);
+                        }
                     }
                 }
             } catch (IOException ex) {

Original file line number	Diff line number	Diff line change
`@@ -460,7 +460,10 @@ private void loadClasses(AuthorizationStack stack, List<File> classfiles, List<F`
`460`	`460`	`}`
`461`	`461`	`// Load the class in memory and try to find a configured space for this class.`
`462`	`462`	`if ((pf = handleLoadClass(classname)) != null) {`
`463`		`- stack.setPlugin(pf);`
	`463`	`+ if (!stack.setPlugin(pf)) {`
	`464`	`+ LOGGER.log(Level.INFO, "plugin {0} is not configured in the stack",`
	`465`	`+ classname);`
	`466`	`+ }`
`464`	`467`	`}`
`465`	`468`	`}`
`466`	`469`
`@@ -475,7 +478,10 @@ private void loadClasses(AuthorizationStack stack, List<File> classfiles, List<F`
`475`	`478`	`}`
`476`	`479`	`// Load the class in memory and try to find a configured space for this class.`
`477`	`480`	`if ((pf = handleLoadClass(classname)) != null) {`
`478`		`- stack.setPlugin(pf);`
	`481`	`+ if (!stack.setPlugin(pf)) {`
	`482`	`+ LOGGER.log(Level.INFO, "plugin {0} is not configured in the stack",`
	`483`	`+ classname);`
	`484`	`+ }`
`479`	`485`	`}`
`480`	`486`	`}`
`481`	`487`	`} catch (IOException ex) {`