hryz / Dapper.Logging

Integration of the DbConnection and Asp.Net Core Logging
MIT License
43 stars 15 forks source link

Log failed commands #11

Closed AndrewP-GH closed 4 years ago

AndrewP-GH commented 4 years ago

Add exception logging with elapsed time for commands execution

aurokk commented 4 years ago

@hryz

hryz commented 4 years ago

@aurokk do you need this functionality? I see that @AndrewP-GH gave up and forked this repo 20 days ago 😁. I can spend a few hours reimplementing this functionality in a way that won't break anyone if you need it.

aurokk commented 4 years ago

Hello, @hryz! Yeah, it's very useful to see logs for failed requests, so it would be nice if you add this functionality :)

hryz commented 4 years ago

OK, I'll do it in a few days (I hope 😉)

hryz commented 4 years ago

The same effect as proposed here is achieved in #12 but without breaking changs. The updated version is here https://www.nuget.org/packages/Dapper.Logging/0.4.3

hryz commented 4 years ago

@aurokk check out the updated package https://www.nuget.org/packages/Dapper.Logging/0.4.3

aurokk commented 4 years ago

@hryz Thank you, works great! 🔥