Open golangisfun123 opened 2 weeks ago
The changes in screener.go
implement new metrics functionality using OpenTelemetry to track blocked and unblocked addresses in the Screener API. This involves importing the go.opentelemetry.io/otel/metric
package, defining a meterName
constant, and adding counters for blocked and unblocked addresses in the screenerImpl
struct.
File Path | Change Summary |
---|---|
contrib/screener-api/screener/screener.go |
Added go.opentelemetry.io/otel/metric import, meterName constant, and metrics counters for tracking blocked and unblocked addresses. |
sequenceDiagram
participant User
participant Screener
participant Metric
User->>Screener: Request address screening
Screener->>Screener: Check if address is blocked
alt Address is blocked
Screener->>Metric: Increment blockedAddressesMetric
Screener->>User: Address blocked response
else Address is unblocked
Screener->>Metric: Increment unblockedAddressesMetric
Screener->>User: Address unblocked response
end
In the realm of code, we see, Metrics bloom, like flowers free. Blocked and unblocked, they do grow, Tracked with care, data flows. Screener’s watch, vigilant and bright, Monitors all with metrics' light. 🌟
[!WARNING]
Review ran into problems
Problems (1)
* Git: Failed to clone repository. Please contact CodeRabbit support.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Latest commit: |
a884926
|
Status: | ✅ Deploy successful! |
Preview URL: | https://56fbda24.sanguine-fe.pages.dev |
Branch Preview URL: | https://add-bl-metrics.sanguine-fe.pages.dev |
Attention: Patch coverage is 50.00000%
with 6 lines
in your changes missing coverage. Please review.
Project coverage is 25.66164%. Comparing base (
15d18e8
) to head (a884926
). Report is 2 commits behind head on master.
Files | Patch % | Lines |
---|---|---|
contrib/screener-api/screener/screener.go | 50.00000% | 4 Missing and 2 partials :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@golangisfun123 did we stage?
Description A clear and concise description of the features you're adding in this pull request.
Additional context Add any other context about the problem you're solving.
Metadata
Summary by CodeRabbit