move initialization logic back to where it used to be

Author: ahmedmuhsin

pom.xml

@@ -21,7 +21,7 @@
     <azure.functions.java.library.version>2.2.0</azure.functions.java.library.version>
     <jupiter.version>5.9.1</jupiter.version>
     <mockito-core.version>4.11.0</mockito-core.version>
-    <appinsights.agents.version>3.7.1</appinsights.agents.version>
+    <appinsights.agents.version>3.6.2</appinsights.agents.version>
   </properties>
   <licenses>
     <license>

src/main/java/com/microsoft/azure/functions/worker/broker/JavaFunctionBroker.java

@@ -68,13 +68,13 @@ public JavaFunctionBroker(ClassLoaderProvider classLoaderProvider) {
 			this.workerObjectCache = null;
 		}
 
-		initializeOneTimeLogics();
 	}
 
 	public void loadMethod(FunctionMethodDescriptor descriptor, Map<String, BindingInfo> bindings)
 			throws ClassNotFoundException, NoSuchMethodException, IOException {
 		descriptor.validate();
 		addSearchPathsToClassLoader(descriptor);
+		initializeOneTimeLogics();
 		FunctionDefinition functionDefinition = new FunctionDefinition(descriptor, bindings, classLoaderProvider);
 
 		if (JAVA_ENABLE_SDK_TYPES_FLAG) {
@@ -112,14 +112,14 @@ private void initializeOneTimeLogics() {
 			synchronized (oneTimeLogicInitializationLock) {
 				if (!oneTimeLogicInitialized) {
 					userContextClassLoader = classLoaderProvider.createClassLoader();
+					oneTimeLogicInitialized = true;
+					initializeFunctionInstanceInjector();
+
 					if (JAVA_ENABLE_SDK_TYPES_FLAG) {
 						loadGlobalMiddlewares();
 					} else {
 						initializeInvocationChainFactory();
 					}
-
-					initializeFunctionInstanceInjector();
-					oneTimeLogicInitialized = true;
 				}
 			}
 		}
@@ -146,15 +146,14 @@ private void initializeInvocationChainFactory() {
 			Thread.currentThread().setContextClassLoader(userContextClassLoader);
 			for (Middleware middleware : ServiceLoader.load(Middleware.class)) {
 				this.serviceLoadedMiddlewares.add(middleware);
-				WorkerLogManager.getSystemLogger().info("Load middleware " + middleware.getClass().getSimpleName());
+				WorkerLogManager.getSystemLogger().info("Load middleware test!!" + middleware.getClass().getSimpleName());
 			}
 		} finally {
 			Thread.currentThread().setContextClassLoader(prevContextClassLoader);
+			ArrayList<Middleware> middlewares = new ArrayList<>(this.serviceLoadedMiddlewares);
+			middlewares.add(getFunctionExecutionMiddleWare(userContextClassLoader));
+			this.invocationChainFactory = new InvocationChainFactory(middlewares);
 		}
-
-        ArrayList<Middleware> middlewares = new ArrayList<>(this.serviceLoadedMiddlewares);
-		middlewares.add(getFunctionExecutionMiddleWare(userContextClassLoader));
-		this.invocationChainFactory = new InvocationChainFactory(middlewares);
 	}
 
 	private void initializeFunctionInstanceInjector() {