fix: http_server.js label names

eggjs / egg-prometheus

Prometheus plugin for Egg.js

https://egg-prometheus.vercel.app

MIT License

39 stars 9 forks source link

Closed jgchenu closed 11 months ago

jgchenu commented 1 year ago

fix: label names

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
egg-prometheus	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 22, 2023 2:55am

jgchenu commented 11 months ago

@fengmk2 得合一下，这两个label没用的，我之前私仓定制魔改的时候提交mr忘记删除了

nightink commented 11 months ago

这两个label没用的

@jgchenu ~对于 /api/:id 这种接口来说如果是想看这个路由情况， path 会比较麻烦吧~

jgchenu commented 11 months ago

这两个label没用的

@jgchenu ~对于 /api/:id 这种接口来说如果是想看这个路由情况， path 会比较麻烦吧~

这个不能上报path的，不然你的metrics 很容易就膨胀了，如果你想要统计的话，应该在defaultMericsFilter方法里面去做处理

nightink commented 11 months ago

这个不能上报path的，不然你的metrics 很容易就膨胀了

@jgchenu 嗯没注意看之前 pr path 已经是被替换成 routerPath 了（本意就是要收集 routerPath 这种的）