bpot / data_fabric

Sharding and replication support for ActiveRecord
http://github.com/mperham/data_fabric
76 stars 11 forks source link

Remove potential leak of config #1

Closed mikepilat closed 12 years ago

mikepilat commented 12 years ago

It's a bit dangerous to dump the full database config (which includes passwords) into the exception message, it ends up in log files and more importantly, things like exception-notification emails where it could be more readily intercepted. Logging just the missing key is enough, IMO.

bpot commented 12 years ago

thanks!