ga4gh / ga4gh-server

Reference implementation of the APIs defined in ga4gh-schemas. RETIRED 2018-01-24
http://ga4gh.org
Apache License 2.0
96 stars 93 forks source link

Revisit server error logging #1105

Open dcolligan opened 8 years ago

dcolligan commented 8 years ago

We decided to merge this pull request before shipping to get errors logged in production:

https://github.com/ga4gh/server/pull/1100

But the downside is that it double-logs errors in development.

(also, we probably shouldn't use the "from" import syntax, for consistency)

david4096 commented 8 years ago

I believe this specific error logging case is covered by https://github.com/ga4gh/server/pull/1154.