Add MissingMethodException'

xuzhg · xuzhg · commit 5ff3ae9ff44a · 2022-06-03T14:25:33.000-07:00
diff --git a/src/Microsoft.AspNet.OData.Shared/DefaultContainerBuilder.cs b/src/Microsoft.AspNet.OData.Shared/DefaultContainerBuilder.cs
@@ -87,8 +87,14 @@ public virtual IServiceProvider BuildContainer()
             {
                 return services.BuildServiceProvider();
             }
-            catch
+            catch (MissingMethodException)
             {
+                /* "services.BuildServiceProvider()" returns IServiceProvider in Microsoft.Extensions.DependencyInjection 1.0 and ServiceProvider in Microsoft.Extensions.DependencyInjection 2.0
+                 * * (This is a breaking change)[https://github.com/aspnet/DependencyInjection/issues/550].
+                * To support both versions with the same code base in OData/WebAPI we decided to call that extension method using reflection.
+                * More info at https://github.com/OData/WebApi/pull/1082
+                */
+
                 MethodInfo buildServiceProviderMethod =
                     typeof(ServiceCollectionContainerBuilderExtensions)
                     .GetMethod(nameof(ServiceCollectionContainerBuilderExtensions.BuildServiceProvider), new[] { typeof(IServiceCollection) });

Original file line number	Diff line number	Diff line change
`@@ -87,8 +87,14 @@ public virtual IServiceProvider BuildContainer()`
`87`	`87`	`{`
`88`	`88`	`return services.BuildServiceProvider();`
`89`	`89`	`}`
`90`		`- catch`
	`90`	`+ catch (MissingMethodException)`
`91`	`91`	`{`
	`92`	`+ /* "services.BuildServiceProvider()" returns IServiceProvider in Microsoft.Extensions.DependencyInjection 1.0 and ServiceProvider in Microsoft.Extensions.DependencyInjection 2.0`
	`93`	`+ * * (This is a breaking change)[https://github.com/aspnet/DependencyInjection/issues/550].`
	`94`	`+ * To support both versions with the same code base in OData/WebAPI we decided to call that extension method using reflection.`
	`95`	`+ * More info at https://github.com/OData/WebApi/pull/1082`
	`96`	`+ */`
	`97`	`+`
`92`	`98`	`MethodInfo buildServiceProviderMethod =`
`93`	`99`	`typeof(ServiceCollectionContainerBuilderExtensions)`
`94`	`100`	`.GetMethod(nameof(ServiceCollectionContainerBuilderExtensions.BuildServiceProvider), new[] { typeof(IServiceCollection) });`