Light Freshness Edit: ASP.NET - HTTP + Native AOT + Kestrel#37046
Light Freshness Edit: ASP.NET - HTTP + Native AOT + Kestrel#37046GitHubber17 wants to merge 4 commits intodotnet:mainfrom
Conversation
|
@GitHubber17, looks like there are some moniker-versining issues. See the details by clicking the OpenPublishing.Build link for details: |
|
@wadepickett - I did look at the Build report. Please see my notes for you in the top comment. Thanks |
Oh, whoops, sorry, I should have read through that first. |
|
"Wade - In the custom components section, the example code is very similar for both moniker ranges. Good question, thanks. It could not work for both versions. The two code samples are fundamentally different because of the hosting model change:
Going forward, I would discontinue consolidating overlap between versions as a goal, and instead focus only on improving the latest version (which will be carried forward). There was almost always a reason for keeping very near-identical code and instructions separate. Differences can look really minor, but any inaccuracy between versions can render that version's content guidance useless and untrustworthy for development. It would also require significant resources to go back and re-test all the older content and code versions against their respective frameworks and packages for what are editorial rather then technical focused updates. |
|
@GitHubber17 , for this question: It should all look like this I think: "## Access HttpContext from middleware" "## Access HttpContext from SignalR" "## Access HttpContext from gRPC methods" "## Access HttpContext from custom components" |
|
@GitHubber17 Correct there is no /Operation folder in the repo. However, the instruction in the article is actually technically correct (but really confusing and could be written better); it is referring refers to a page route, not a file system folder. Good catch, it does need help. Pages/Operation.cshtml is a file in the project, and /Operation is the route (URL) a developer navigates to in the browser when running the app. They're connected by convention, no explicit route registration is needed. The original article text was confusing. When it said "go to the /Operation folder," there's no folder, there's a Pages/Operation.cshtml file that produces a /Operation route. The article meant "navigate to https://localhost:{PORT}/Operation in your browser." How to correct this: I will add the suggestion inline. |

This PR contains light freshness updates to the following articles:
Access HttpContext in ASP.NET Core
moniker range >= 6.0,but article descriptions for both ranges say SignalR and gRPC are supported.
Should the two sections be visible for both moniker ranges?
Could the example for the
>=6.0range work for both article versions?Make HTTP requests using IHttpClientFactory in ASP.NET Core
moniker range >= 6.0only; Otherwise, updated for Acrolinx"In the HttpRequestsSample download, go to the /Operation folder..."
ASP.NET Core support for Native AOT
Configure endpoints for the ASP.NET Core Kestrel web server
<xref>link syntax for broken link, here is the URL<xref:Microsoft.AspNetCore.Server.Kestrel.KestrelConfigurationLoader.Endpoint...
For details, see ADO request 568764.
[ Note from Wade: I updated this to address by myself and Tom, since this is Tom's content. ]
Internal previews