initial commit for the elm logger

[sonjakhan]

A basic implementation of ELM, a wannabe ELMAH. Currently the logs are written to a dictionary and are displayed in plain text

[borgdylan]

Would it be possible to make the function that writes the dictionary out to the response replaceable in ElmOptions? Like that, I could add transform it into JSON and integrate it into some kind of management dashboard.

[kevinchalet]

Throwing an exception here will make MVC unusable as it relies on BeginScope for its route handler and the default action selector.

If you don't wanna implement it yet, I'd suggest returning either null or a dummy IDisposable (returning null shouldn't be an issue given that BeginScope is only used with using, but a custom IDisposable is probably better).

[sonjakhan]

thanks for the catch!

[sonjakhan]

@borgdylan I am working on abstracting the storage system of the logs

[borgdylan]

@sonjakhan that would help a lot since I do all my UI in AngularJS+WInJS which means data is fetched as JSON.

[Eilon]

I think you can initialize the property inline. Something like this:

public static List<string> Log { get; set; } = new List<string>();

[kevinchalet]

Removing the setter would be even better 🎉

[davidfowl]

This is all being removed anyways 😸

[Eilon]

Baby steps 😋

[sonjakhan]

This is pretty ugly right now and doesn't show all the data that's stored. Grouping by request, filtering by severity, sorting by time, etc are things I was thinking about implementing but I wanted to get some feedback before I went too far down a bad path. Linking to the full details of a request is also important - currently I store whatever is in LogContext, which is probably missing a couple of properties.

[kevinchalet]

I imagine that you could also use an instance store and declare ElmStore as singleton instead of transient.

[sonjakhan]

new interface:

filtering by logger name:

highlight scope on hover:

path link shows request details and logs:

[Tratcher]

1. Elm needs to be moved to the Diagnostics repo due to its dependencies.
2. Remove the MVC dependency from the sample. You can use this Hello World code instead:
   https://github.com/aspnet/Entropy/blob/dev/samples/Builder.HelloWorld.Web/Startup.cs

[sonjakhan]

Working on fixes to scoping, and will reopen a pull request in the Diagnostics repo when finished.