vechain / thor

A general purpose blockchain highly compatible with Ethereum's ecosystem

GNU Lesser General Public License v3.0

799 stars 248 forks source link

use middleware to capture metrics #788

Closed libotony closed 1 month ago

libotony commented 1 month ago

Description

This PR changed the active websocket count to be captured in the metrics middleware, and removed duration metrics for websockets.

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] New and existing E2E tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have not added any vulnerable dependencies to my code