AppSettings.json to configure Serilog.File Sink doesn't seem to work

[mistyn8]

Which exact Umbraco version are you using? For example: 8.13.1 - don't just write v8

v9-rc002

Bug summary

 "Serilog": {
    "MinimumLevel": {
      "Default": "Information"
    },
    "WriteTo": [
      {
        "Name": "Async",
        "Args": {
          "configure": [
            {
              "Name": "Console"
            },
            {
              "Name": "File",
              "Args": {
                "restrictedToMinimumLevel": "Warning"
              }
            }            
          ]
        }
      }      
    ]
  },

I'm expecting that the json files only now write warning logs, but information logs are still appended.

Specifics

Ps I'm wanting to do this as Using Seq Sink for better structured logging/reporting/interrogation.. so would like information logging there, so only require minimal logging in json files.

https://github.com/serilog/serilog/wiki/Configuration-Basics#overriding-per-sink

Setting the minimumLevel for serilog globally would affect all sinks.

https://github.com/umbraco/Umbraco-CMS/blob/48fcfcae21c579231268147167619274482abef3/src/Umbraco.Infrastructure/Logging/Serilog/LoggerConfigExtensions.cs presumably is where config needs to take account of appsettings.json

Steps to reproduce

update appsettings.json as in the summary, and json file based logs are not restricted to Warning.

Expected result / actual result

No response

[mistyn8]

Ps this might just be that the name isn't File for the minimalConfiguration logging? But can't find any documentation elsewhere.

and

Umbraco-CMS/src/Umbraco.Infrastructure/Logging/Serilog/LoggerConfigExtensions.cs

Line 41 in 48fcfca

logConfig.MinimumLevel.Verbose() //Set to highest level of logging (as any sinks may want to restrict it to Errors only)

Implies we might want to and should be able to set for other sinks?

[bergmania]

Hi @mistyn8 ..

I can confirm this is not possible. Ill make a PR to make it possible. As the UmbracoFile is not using a regular File Sink due to performance concerns, It will be something like using a "UmbracoFile" name, like this example

{
  "Serilog": {
    "MinimumLevel": {
      "Default": "Information",
    },
    "WriteTo": [
      {
        "Name": "UmbracoFile",
        "Args": {
          "restrictedToMinimumLevel": "Warning"
        }
      },
      {
        "Name": "Async",
        "Args": {
          "configure": [
            {
              "Name": "Console",
              "Args": {
                "restrictedToMinimumLevel": "Information"
              }
            }
          ]
        }
      }
    ]
  },
}