search

hryz / Dapper.Logging

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

Add command parameters to the log #2

Closed levilxw closed 5 years ago

levilxw commented 5 years ago
  1. Add command parameters to the log
  2. Upgrade the dependancy packages version
hryz commented 5 years ago

Hi @levilxw. Thank you for the PR! However, in the current form, it can't be accepted. The query parameters might contain sensitive data (Personal Identifiable Information). I'm going to check out your code and improve it. There will be a configuration setting for this. By default, it will be disabled (just like in EF Core). Stay tuned!