datalust / seq-app-mail Goto Github PK

See datalust/seq-app-htmlemail#89 and datalust/seq-app-htmlemail#112.

Currently, alerts are not being detected and so use the default property name/value style template:

The template does already have support for alerts, brought over from the current app, so guessing this is a problem with the condition.

Not sure this is what you want? The token is in the appveyor file here in GitHub?

• @app.... built-in properties
• Default and supplied time zone/date format support
• first template repetition support
• Improve default email subject
• Publishing
• Document the template syntax
• Screenshot of the default template in README
• Package icon
• Package description etc.
• Link CI build and NuGet packages from README

The project has been bootstrapped in part from the Serilog.Expressions codebase; this has left us with some oddities, such as:

• @Level is always a Serilog LogEventLevel name (original levels are converted)
• TypeOf() does not return the same set of values that Seq's built-in TypeOf() does

This list is sure to grow; while we will ideally work around them/paper them over in the short term, longer term we should be looking to switch the core data model from LogEvent and LogEventPropertyValue to e.g. System.Text.Json's JsonDocument and JsonElement.

There are other "beneficial" differences, such as the use of DateTimeOffset for datetime types enabling richer formatting than Seq's built-in datetimes can provide, and @l supporting the richer level formatting that Serilog does - so it's worth looking for/giving some thought to these before going too far down this path.

Name resolution happens too late, so we don't end up producing a CompiledLevelToken. Some rethinking required.

This will allow it to be consumed downstream, e.g. by Seq.App.HttpRequest and other packages.

(We should consider publishing Seq.Mail soon, too, to support third-party mail system integrations.)