feat: better organized request logs

cofacts / rumors-api

GraphQL API server for clients like rumors-site and rumors-line-bot

https://api.cofacts.tw

MIT License

111 stars 26 forks source link

feat: better organized request logs #305

Closed MrOrz closed 1 year ago

MrOrz commented 1 year ago

This is part of the operation layer enhancement; see https://g0v.hackmd.io/luIa-z9HToC2zlFpD9yK3w#Op-%E5%88%86%E8%BA%AB%E5%B8%B3%E8%99%9F%E8%99%95%E7%90%86 .

Reorganize the logs in logging mechanism

records HTTP header for GraphQL requests
- Also logs operation name, query, variables
- Enables tracking operations of a certain IP address
Compact GraphQL logging
- Long strings are truncated with "..."

Screenshot

First log comes from GraphQL plugin. The next log comes from pino logger.

coveralls commented 1 year ago

coverage: 88.394% (-0.07%) from 88.468% when pulling 3db4125569d6fd6f269270a6337fa45688ffb4b8 on logging into 9b3d6964e5a9656418d255b462a1c361c2d19ede on master.