Skip to content

Fix Request Logging link in Minimal APIs middleware table#36888

Merged
wadepickett merged 9 commits into
mainfrom
copilot/fix-request-logging-link
Mar 18, 2026
Merged

Fix Request Logging link in Minimal APIs middleware table#36888
wadepickett merged 9 commits into
mainfrom
copilot/fix-request-logging-link

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 17, 2026

The "Request Logging" row in the Minimal APIs middleware table links to the general logging article (xref:fundamentals/logging/index) instead of the HTTP Logging middleware article (xref:fundamentals/http-logging/index). The description and API column (UseHttpLogging) both clearly refer to HTTP Logging middleware, confirming the link target was simply wrong. Present across all version monikers (6.0–10.0).

  • Link fix across 5 files: xref:fundamentals/logging/indexxref:fundamentals/http-logging/index in minimal-apis.md and include files for versions 6–9
  • Remove stale version pin: Dropped ?view=aspnetcore-6.0 query parameter from the corrected links in include files
  • Update ms.date in minimal-apis.md
Original prompt

This section details on the original issue you should resolve

<issue_title>Minimal APIs quick reference for .net 8 - Request Logging link leads to incorrect page</issue_title>
<issue_description>### Description

In the section ASP.NET Core Middleware clicking on Request Logging link leads to a general logging topic for ASP.NET Core 6.0

I believe it should lead to a respective HTTP Logging middleware topic at HTTP logging in ASP.NET Core

Image

Page URL

https://learn.microsoft.com/en-us/aspnet/core/fundamentals/minimal-apis?view=aspnetcore-8.0

Content source URL

https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/fundamentals/minimal-apis.md

Document ID

b09824ab-b051-75a1-13b7-b0e281cc46e0

Platform Id

24c01f0e-630a-08a5-698d-9668e1a7b40e

Article author

@wadepickett

Metadata

  • ID: b09824ab-b051-75a1-13b7-b0e281cc46e0
  • PlatformId: 24c01f0e-630a-08a5-698d-9668e1a7b40e
  • Service: aspnet-core
  • Sub-service: fundamentals

Related Issues</issue_description>

<agent_instructions>Read both the issue description and the discussion. The discussion contains important instruction suggestions for fixing this issue across topic versions. Also make sure the ms.date on the updated topic is reset to today's date in the proper format</agent_instructions>

Comments on the Issue (you are @copilot in this section)

@wadepickett Thanks @AFayziev! You are correct, the link needs to be changed. The link specified a version and that was left in from versions 7 through 9. We will fix it. @wadepickett ## 🤖 AI Triage Summary

📌 Note to community: This is an automated preliminary analysis to help our documentation team quickly review, determine scope and prioritize this issue. This report is not a resolution or answer to your question—it's an internal triage tool that identifies potentially relevant docs, code samples, and versions to look into. A team member will review this issue and respond. Thank you for your contribution!


This preliminary assessment report was run by: @wadepickett
Date: 2026-03-17
Issue: 36884
Model: GitHub Copilot


Issue Analysis: Request Logging link in Minimal APIs middleware table points to incorrect page

✅ Issue Validation

Status: Valid and actionable

📋 Issue Summary

In the "ASP.NET Core Middleware" table of the Minimal APIs quick reference article, the Request Logging row links to the general ASP.NET Core logging topic (xref:fundamentals/logging/index) instead of the HTTP Logging middleware topic (xref:fundamentals/http-logging/index). This is a mismatch: the description says "Provides support for logging HTTP requests and responses" and the API column references UseHttpLogging, both of which clearly refer to HTTP Logging middleware — not the general logging infrastructure. This incorrect link exists across all version monikers (6.0, 7.0, 8.0, 9.0, and 10.0).

📁 Potentially Affected Files

These files have been identified as possibly related to this issue and may need review.

File Path Lines Section
Main article (10.0) aspnetcore/fundamentals/minimal-apis.md 53 "ASP.NET Core Middleware" table
Include (9.0) aspnetcore/fundamentals/minimal-apis/includes/minimal-apis9.md 28 "ASP.NET Core Middleware" table
Include (8.0) aspnetcore/fundamentals/minimal-apis/includes/minimal-apis8.md 28 "ASP.NET Core Middleware" table
Include (7.0) aspnetcore/fundamentals/minimal-apis/includes/minimal-apis7.md 28 "ASP.NET Core Middleware" table
Include (6.0) [aspnetcore/fundamentals/minimal-apis/includes/minimal-apis6.md](https://github.com/dotnet/AspN...

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.


Internal previews

📄 File 🔗 Preview link
aspnetcore/fundamentals/minimal-apis.md Minimal APIs quick reference

…versions

Changes xref:fundamentals/logging/index to xref:fundamentals/http-logging/index
in the Minimal APIs middleware table for versions 6.0-10.0.
Also removes hardcoded ?view=aspnetcore-6.0 query parameter.

Fixes #36884

Co-authored-by: wadepickett <10985336+wadepickett@users.noreply.github.com>
Copilot AI changed the title [WIP] [ISSUE-1] Fix Request Logging link in minimal APIs reference Fix Request Logging link in Minimal APIs middleware table Mar 17, 2026
Copilot AI requested a review from wadepickett March 17, 2026 21:54
@wadepickett wadepickett marked this pull request as ready for review March 17, 2026 22:45
Comment thread aspnetcore/fundamentals/minimal-apis/includes/minimal-apis6.md Outdated
Comment thread aspnetcore/fundamentals/minimal-apis/includes/minimal-apis7.md Outdated
Comment thread aspnetcore/fundamentals/minimal-apis/includes/minimal-apis7.md Outdated
Comment thread aspnetcore/fundamentals/minimal-apis/includes/minimal-apis9.md Outdated
Comment thread aspnetcore/fundamentals/minimal-apis/includes/minimal-apis8.md Outdated
Minor fixes for missing periods and commas.

Co-authored-by: Wade Pickett <wpickett@microsoft.com>
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.

Approved.

Copy link
Copy Markdown
Contributor

@tdykstra tdykstra left a comment

Choose a reason for hiding this comment

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

I have never liked that "for experienced developers" label. To me it sounds like a disguised excuse for poorly written content. This article is really a kind of reference content and should be accessible to almost anyone. I'm not saying you should do anything now but let's keep this in mind for trials of Copilot rewrites to improve readability. 😄

Updated the introduction to clarify the purpose of the document and removed redundant bullet points.
Updated the introductory text for Minimal APIs documentation.
Removed redundant points and clarified the document's purpose.
Updated the description of the Minimal APIs document to clarify its purpose and target audience.
@wadepickett wadepickett merged commit ffcb03b into main Mar 18, 2026
4 checks passed
@wadepickett wadepickett deleted the copilot/fix-request-logging-link branch March 18, 2026 04:40
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.

Minimal APIs quick reference for .net 8 - Request Logging link leads to incorrect page

3 participants