monarch-initiative / biolink-api

API for linked biological knowledge
https://api.monarchinitiative.org/api/
BSD 3-Clause "New" or "Revised" License
63 stars 25 forks source link

Include more upstream information in logging to aid while analyzing logs #220

Open deepakunni3 opened 6 years ago

deepakunni3 commented 6 years ago

Include more upstream information to facilitate analysis of logs and study traffic.

This might require changes on the ontobio side as well. Will update with more information as we move forward.

@kltm Feel free to add more information and what you would like to see in the logs.

kltm commented 6 years ago

Thank you @deepakunni3 !

I think things that might be of interest would be:

As it stands right now, one must try and match entries in nginx and monarch-api, which have very different ideas of logging (e.g. one line per call vs. debug stream). It may be good to just have a high-level summary line for each call in the monarch app, and some method of trying those to individual sub-calls needed to perform it. Alternatively, every log line could contain all of the information necessary.