Feature request: Run analyzers in debug mode

[Rick-Anderson]

Problem motivation: #18768, app.UseSpa was in the wrong order.

Add app.UseRunAllAnalyzers to template code to detect middleware order and other problems.

public void Configure(IApplicationBuilder app, IWebHostEnvironment env)
{
    if (env.IsDevelopment())
    {
        app.UseDeveloperExceptionPage();
        app.UseDatabaseErrorPage();
        app.UseRunAllAnalyzers();   // run middleware order check and other checks
    }
    else
    {
        app.UseExceptionHandler("/Error");
        app.UseHsts();
    }

    ...

UseRunAllAnalyzers could be updated frequently to add more checks.

See https://twitter.com/davidfowl/status/1228902288627912705 where @davidfowl requests this.

cc @javiercn

[javiercn]

@Rick-Anderson thanks for following up.

@mkArtakMSFT I'm moving this to next milestone planning so that we can look at it then and decide.

[Rick-Anderson]

See #20709, caused by middleware in the wrong order.

[Rick-Anderson]

How to use Static Files with CatchAll Routing? #18038
Yet another issue that could have been detected.

[mkArtakMSFT]

We've moved this issue to the Backlog milestone. This means that it is not going to be worked on for the coming release. We will reassess the backlog following the current release and consider this item at that time. To learn more about our issue management process and to have better expectation regarding different types of issues you can read our Triage Process.

[Rick-Anderson]

For future reference, here's an easy one to miss.

UseHsts depends on the request scheme being updated, therefore UseForwardedHeaders must be called before UseHsts.

[Rick-Anderson]

Here's another
Order of UseStatusCodePagesWithReExecute and UseRouting shouldn't matter #19374