Support adding default tags as discrete values in LoggerConfiguration…

[peef]

… extension

Motivation

When Exceptionles sink is used together with serilog-settings-configuration package then it can be configured in appsettings.json:

"Serilog": {
    "WriteTo": [
        {
            "Name": "Exceptionless",
            "Args": {
                "apiKey": "<my-api-key>"
            }
        }
    ]
}

with the configuration code:

loggerConfiguration.ReadFrom.Configuration(cfg);

In this case it is not possible to add default tags to the sink.

The PR adds the possibility to configure default tags in appsettings.json:

"Serilog": {
    "WriteTo": [
        {
            "Name": "Exceptionless",
            "Args": {
                "apiKey": "<my-api-key>",
                "defaultTags": ["tag1", "tag2"]
            }
        }
    ]
}

[niemyjski]

Thanks for the PR! It looks good, we can simplify the add tags check and remove linq :).

[peef]

If the PR will be accepted and merged, could be new version of the nuget package released, please?

[niemyjski]

Thanks for the pr

[ejsmith]

I was looking at this and thinking it would be good to all DefaultData to be added to as well, but then I thought... this logging sink has to be using an ExceptionlessClient instance under the hood... why don't we just configure these things on the client? I guess it doesn't hurt anything and make it more discoverable to have them here as well.

[peef]

Thank you for merging. I can see there is new version v89 released, but the nupkg asset has still old version (3.1.3). Could the new version (3.1.4) be released also to NuGet public gallery, please?

[peef]

@ejsmith I can see the sink creates new instance of the ExceptionlessClient, see the code. AFAIK there is only way how to add default tags: via default ExceptionlessClient.Default instance. So in the case of this PR it is probably not possible to add default tags to the under hood client. But correct me if I'm wrong, please.

BTW I cannot see the issue tracker in this project, but maybe it would be nice having it for similar discussions.

[niemyjski]

I just pushed a new version.