Paymentsense / Dapper.SimpleLoad

Dapper.SimpleLoad
MIT License
13 stars 4 forks source link

Query warnings should emit stack trace and query #10

Closed bartread closed 7 years ago

bartread commented 8 years ago

It's nice to know that a query took a long time. For example:

2016-01-08 14:42:46,150 [57] WARN  Dapper.SimpleLoad.SimpleLoadExtensions [(null)] - RECEIVED 1 ROWS IN QUERY RESULT SET IN 17235ms.

However, this warning would be a lot more useful if we knew where it had come from in the codebase, and what SQL was executed; possibly even the values of parameters.

bartread commented 7 years ago

This was fixed a long while ago. We don't emit the parameter values because that's something of of a security risk (parameters could contain data that should remain confidential).