Aspnetcore.docs: Consistent stdout log naming needed

Created on 2 Dec 2016  路  7Comments  路  Source: dotnet/AspNetCore.Docs

There are inconsistent stdout log names in the docs. I've spotted a few here and there, and I see it again in the Response Caching sample ...

stdoutLogFile=".\logs\stdout"

It should be ...

stdoutLogFile=".\logs\aspnetcore-stdout"

I propose to submit a PR that will make these consistent and correct across the docs. Sound good?

Most helpful comment

Maybe it's more a product than a docs issue and we look at changing the VS project templates to create ".\logsaspnetcore-stdout"? Docs would then follow that lead.

All 7 comments

@spboyer @tdykstra please review proposal

Why do you think it "should be" aspnetcore-stdout? When writing stdout in node, it is not nodejs-stdout.

I'd prefer to use what the VS project templates create, which is what you see in the Response Caching sample, i.e., I would go in the opposite direction from what you're proposing.

https://github.com/aspnet/AspNetCoreModule/blob/93ace1b84bd79382233cb39b858611d2db05552e/src/AspNetCore/aspnetcore_schema.xml#L24

[EDIT] Also consistent with stdoutLogFile in https://www.iis.net/learn/extensions/httpplatformhandler/httpplatformhandler-configuration-reference

[EDIT] No strong preference on my part; although, I do prefer to see samples that match the config default. I'm mainly supporting consistency here.

Maybe it's more a product than a docs issue and we look at changing the VS project templates to create ".\logsaspnetcore-stdout"? Docs would then follow that lead.

changing the VS project templates to create ".\logsaspnetcore-stdout"
If others concur I'll open a template issue.

@GuardRex we've decided to not change the names at this time.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

wgutierrezr picture wgutierrezr  路  3Comments

StevenTCramer picture StevenTCramer  路  3Comments

Raghumu picture Raghumu  路  3Comments

royshouvik picture royshouvik  路  3Comments

Rick-Anderson picture Rick-Anderson  路  3Comments