URL rewrite doesn't work for non-regex about basicmiddleware

[Tratcher]

So I followed the docs and did this in Startup.cs at the end of Configure() (after app.UseMvc(), I hope that's correct?):

Rewrite must come early in the pipeline, not at the end. In your case it needs to come before UseStaticFiles.

[mkArtakMSFT]

Thanks for contacting us, @lonix1.
Please provide us with a minimalistic repro project so we can investigate the issue.

[mkArtakMSFT]

From @kichalla on August 10, 2018 18:30

cc @jkotalik

[mkArtakMSFT]

From @jkotalik on August 10, 2018 18:32

Seems like a bug, will look into it in the 2.2 timeframe.

[mkArtakMSFT]

@jkotalik, should this be moved to a different repo?

[mkArtakMSFT]

From @jkotalik on August 10, 2018 23:16

Oops: just BasicMiddleware: https://github.com/aspnet/BasicMiddleware

[mkArtakMSFT]

From @gsdnano on August 11, 2018 3:28

This can be reproduced by taking the example at https://docs.microsoft.com/en-us/aspnet/core/fundamentals/url-rewriting?view=aspnetcore-2.1&tabs=aspnetcore2x and commenting out the app.Run command at the end of the Configure function. The url rewrite example will then 404 no matter the replacement url. This is using core 2.1 and Microsoft.AspNetCore.Rewrite 2.1.1.

[mkArtakMSFT]

From @lonix1 on August 11, 2018 6:0

Once this bug is resolved, please also give us some guidance (or better yet, update the docs) about the implications of putting this before/after/between app.UseStaticFiles() and app.UseMvc(), or whether it should be at the beginning/end of Configure() i.e. the beginning/end of the pipeline.

[mkArtakMSFT]

@muratg FYI

[jkotalik]

@lonix1 I assume this solved your issue. Closing. You may reopen if you are still having issues.