gofr-dev / gofr-dev/gofr

Refactor metrics implementation

Open
#629 4 comments 0 reactions 0 assignees View on GitHub
help wanted Observability
Dominant language
Go
Stars
20.9k
Forks
1.8k
Avg merge
5d 18h
Merged PRs (30d)
39

Description

Extension of #529
- A new counter metric needs to be added which represents server errors (`app_server_error`)
- A new counter metric for circuit open counts (`app_circuit_open_count`)

Current metrics implementation makes it harder to use in different packages as it requires an instance of metricsManager everywhere, need to think of a better solution and implement in such a way that a copy of metricsManager should not be required to pass everywhere.

Contributor guide

Open the contributing guide

Research direction

Start with the current metrics implementation and issue #529, then trace how metricsManager is passed across packages. Define the approach for shared metric access, add the app_server_error and app_circuit_open_count counters, and verify that callers no longer need to pass a copy of metricsManager everywhere.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
observability
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.