Tweak WebpackDevMiddleware.cs to avoid unnecessary Task.Yield()

SteveSandersonMS · SteveSandersonMS · commit 6d8767d14158 · 2016-08-18T13:49:20.000-07:00
diff --git a/src/Microsoft.AspNetCore.SpaServices/Webpack/WebpackDevMiddleware.cs b/src/Microsoft.AspNetCore.SpaServices/Webpack/WebpackDevMiddleware.cs
@@ -81,12 +81,12 @@ public static void UseWebpackDevMiddleware(
             // sees, not "localhost", so that it works even when you're not running on localhost (e.g., Docker).
             appBuilder.Map(WebpackHotMiddlewareEndpoint, builder =>
             {
-                builder.Use(next => async ctx =>
+                builder.Use(next => ctx =>
                 {
                     var hostname = ctx.Request.Host.Host;
                     ctx.Response.Redirect(
                         $"{WebpackDevMiddlewareScheme}://{hostname}:{devServerInfo.Port.ToString()}{WebpackHotMiddlewareEndpoint}");
-                    await Task.Yield();
+                    return Task.FromResult(0);
                 });
             });
         }

Original file line number	Diff line number	Diff line change
`@@ -81,12 +81,12 @@ public static void UseWebpackDevMiddleware(`
`81`	`81`	`// sees, not "localhost", so that it works even when you're not running on localhost (e.g., Docker).`
`82`	`82`	`appBuilder.Map(WebpackHotMiddlewareEndpoint, builder =>`
`83`	`83`	`{`
`84`		`- builder.Use(next => async ctx =>`
	`84`	`+ builder.Use(next => ctx =>`
`85`	`85`	`{`
`86`	`86`	`var hostname = ctx.Request.Host.Host;`
`87`	`87`	`ctx.Response.Redirect(`
`88`	`88`	`$"{WebpackDevMiddlewareScheme}://{hostname}:{devServerInfo.Port.ToString()}{WebpackHotMiddlewareEndpoint}");`
`89`		`- await Task.Yield();`
	`89`	`+ return Task.FromResult(0);`
`90`	`90`	`});`
`91`	`91`	`});`
`92`	`92`	`}`