feat: record less bytes in GraphQL request

cofacts / rumors-api

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

https://api.cofacts.tw

MIT License

109 stars 26 forks source link

feat: record less bytes in GraphQL request #312

Closed MrOrz closed 10 months ago

MrOrz commented 10 months ago

Only record IP related header for easier user tracking
- headers makes ~70% of the log length
- other headers are already captured by pino
remove spaces GraphQL query and reduce string length cap
Removes variables from GraphQL to exclude sensitive data.

coveralls commented 10 months ago

coverage: 88.223% (+0.005%) from 88.218% when pulling e12a71649b7330d8b8525bc10d440224fe0d6ff7 on compress-logs into 9a72f6250daa235ba2c9e648f0dae300ab5ada9b on master.