Skip to content

Light Freshness Edit: ASP.NET - host and deploy#37034

Open
lootle1 wants to merge 13 commits intodotnet:mainfrom
lootle1:FR47
Open

Light Freshness Edit: ASP.NET - host and deploy#37034
lootle1 wants to merge 13 commits intodotnet:mainfrom
lootle1:FR47

Conversation

@lootle1
Copy link
Copy Markdown

@lootle1 lootle1 commented Apr 20, 2026

This PR contains freshness updates to the following articles:

  • ASP.NET Core Module (ANCM) for IIS
  • Common error troubleshooting for Azure App Service and IIS with ASP.NET Core
  • Advanced configuration of the ASP.NET Core Module and IIS
  • Host ASP.NET Core on Windows with IIS
  • Host and deploy ASP.NET Core
  • web.config file
  • Configure ASP.NET Core to work with proxy servers and load balancers
    Line 314, the link for NGINX: Using the Forwarded header redirects to a generic docs page. I replaced with this article link but wanted to confirm this is what you'd like to replace.

Minor grammar/style updates. Link path updates.

For request details, see 568764


Internal previews

📄 File 🔗 Preview link
aspnetcore/host-and-deploy/aspnet-core-module.md ASP.NET Core Module (ANCM) for IIS
aspnetcore/host-and-deploy/azure-iis-errors-reference.md Common error troubleshooting for Azure App Service and IIS with ASP.NET Core
aspnetcore/host-and-deploy/iis/advanced.md Advanced configuration
aspnetcore/host-and-deploy/iis/index.md Host ASP.NET Core on Windows with IIS
aspnetcore/host-and-deploy/iis/web-config.md aspnetcore/host-and-deploy/iis/web-config
aspnetcore/host-and-deploy/index.md Host and deploy ASP.NET Core
aspnetcore/host-and-deploy/proxy-load-balancer.md Configure ASP.NET Core to work with proxy servers and load balancers

@wadepickett
Copy link
Copy Markdown
Contributor

Hi @lootle1, I will provide a review as soon as I see this is no longer in draft and intented for review or you ping me for one. Thanks for the contribution!

@wadepickett wadepickett changed the title Light Freshness Edit: ASP.NET Light Freshness Edit: ASP.NET - host and deploy Apr 20, 2026
@wadepickett
Copy link
Copy Markdown
Contributor

Extending the name of the PR with a bit of detail since we had more than one by the same name.

@lootle1
Copy link
Copy Markdown
Author

lootle1 commented Apr 21, 2026

@dotnet-policy-service agree company="Microsoft"

@lootle1
Copy link
Copy Markdown
Author

lootle1 commented Apr 21, 2026

@lootle1 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.

@dotnet-policy-service agree [company="{your company}"]

Options:

  • (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@dotnet-policy-service agree
  • (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@dotnet-policy-service agree company="Microsoft"

Contributor License Agreement

@dotnet-policy-service agree company="Microsoft"

@wadepickett
Copy link
Copy Markdown
Contributor

wadepickett commented Apr 22, 2026

@lootle1, not sure how many files you intend to drop in this PR, but I think 5 is probibly good. We add too many articles with significant changes per PR and it becomes more problematic to revert, or address merge conflict issues in main or later pushed to live. If that messes too much with your process and cuts down on your work groove, I can go with more at a time if its worth the trade off.

@lootle1
Copy link
Copy Markdown
Author

lootle1 commented Apr 23, 2026

@wadepickett ah sounds good! I was continuing to add to have all the host-and-deploy articles in one PR. I'll stop with this last one I'm working on for this PR and stick with the five articles per moving forward, that's no problem! Overall finding these are in good shape so mostly link replacements and grammar/style updates. Hopefully fairly quick to review!

@lootle1 lootle1 marked this pull request as ready for review April 23, 2026 20:56
@lootle1
Copy link
Copy Markdown
Author

lootle1 commented Apr 23, 2026

@tdykstra @wadepickett this PR is ready for review. Top comment has an overview of the seven articles. I have one specific call out for a link but other than that should be fairly straightforward I believe. Let me know if any change requests, thank you!

Comment thread aspnetcore/host-and-deploy/proxy-load-balancer.md Outdated
Comment thread aspnetcore/host-and-deploy/proxy-load-balancer.md Outdated
Comment thread aspnetcore/host-and-deploy/aspnet-core-module.md Outdated
Comment thread aspnetcore/host-and-deploy/aspnet-core-module.md Outdated
Comment thread aspnetcore/host-and-deploy/aspnet-core-module.md Outdated
Comment thread aspnetcore/host-and-deploy/aspnet-core-module.md Outdated
Comment thread aspnetcore/host-and-deploy/iis/index.md Outdated
Comment thread aspnetcore/host-and-deploy/proxy-load-balancer.md Outdated
Copy link
Copy Markdown
Contributor

@wadepickett wadepickett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lootle1, great work! I made some suggestions, which you can commit if you want to use them as-is. There were only two medium issues a couple very minor ones.

@wadepickett wadepickett requested a review from tdykstra April 24, 2026 19:31
lootle1 and others added 6 commits April 27, 2026 08:41
Co-authored-by: Wade Pickett <wpickett@microsoft.com>
Co-authored-by: Wade Pickett <wpickett@microsoft.com>
Co-authored-by: Wade Pickett <wpickett@microsoft.com>
Co-authored-by: Wade Pickett <wpickett@microsoft.com>
Co-authored-by: Wade Pickett <wpickett@microsoft.com>
Co-authored-by: Wade Pickett <wpickett@microsoft.com>
lootle1 and others added 2 commits April 27, 2026 08:44
Co-authored-by: Wade Pickett <wpickett@microsoft.com>
Co-authored-by: Wade Pickett <wpickett@microsoft.com>
@lootle1
Copy link
Copy Markdown
Author

lootle1 commented Apr 27, 2026

@wadepickett thanks, all those changes looked good to me so committed. Unless @tdykstra has any change requests this PR should be all set. Appreciate you both!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants