Closed trajan0x closed 1 week ago
The recent code changes involve various updates, including changes to import paths, function parameters, and dependencies. Key adjustments include the switch from instrumentation
to httpcapture
for HTTP transport capturing, introduction of new middleware, restructuring of testing suites, and the addition of new metrics. Modifications also extend to adding new dependencies, improving CORS configurations, and enhancing command functionality for a Slack bot.
Files | Change Summary |
---|---|
contrib/promexporter/exporters/exporter.go |
Updated import paths and replaced usage of instrumentation.NewCaptureTransport with httpcapture.NewCaptureTransport |
contrib/promexporter/go.mod |
Removed dependency on github.com/google/go-cmp v0.6.0 |
contrib/promexporter/internal/gql/explorer/contrib/main.go |
Replaced util2.WaitForStart with ginhelper.WaitForStart |
core/ginhelper/* |
Added server timeout management, updated context handling for HTTP requests, and modified CORS configuration using constants |
core/go.mod |
Added indirect dependencies on github.com/stretchr/objx and go.opentelemetry.io/otel/exporters/stdout/stdoutmetric |
core/metrics/instrumentation/httpcapture/* |
Package comment added, function updated to include handler parameter, and new test suite HTTPCaptureSuite |
core/metrics/metrics.go |
Added //go:generate directive for mock generation and updated Gin() method in Handler interface to return a slice |
core/metrics/instrumentation/otelginmetrics/* |
Added Recorder interface for HTTP metrics and functions to retrieve versions |
ethergo/submitter/* |
Added gas balance metrics, new functions for recording balance, and updated related structures |
.golangci.yml |
Skipped linting for otelginmetrics/* directory |
contrib/opbot/* |
Added signozEnabled field, methods for middleware and commands, and new dependencies |
core/metrics/mocks/handler.go |
Introduced autogeneration of mock Handler with various mock methods |
core/metrics/tester.go |
Added helper functions to retrieve information from tracetest.SpanStub for testing |
core/bytes_test.go |
Iterated over test cases using an index variable i |
core/metrics/instrumentation/otelginmetrics/recorder.go |
Introduced Recorder interface for tracking and observing HTTP metrics |
core/metrics/instrumentation/otelginmetrics/version.go |
Added functions to retrieve release version and semantic version |
contrib/opbot/botmd/* |
Added signozEnabled field and functions for handling Signoz configuration in commands and middleware |
core/metrics/null.go |
Modified Gin method in nullHandler to return a slice of gin.HandlerFunc |
contrib/scribe-api/scribe/scribe.go |
Modified router.Use call in NewScribe to use variadic arguments |
In the realm of code so bright, Changes bloom like stars at night. ✨ Metrics hum a rhythmic song, Tracing paths where bytes belong. Middleware ascends with grace, Handling tasks in every space. Rabbits cheer with joyful hue, Code evolves, refreshed, anew. 🌼
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: |
661c235
|
Status: | ✅ Deploy successful! |
Preview URL: | https://3f50ebb5.sanguine-fe.pages.dev |
Branch Preview URL: | https://fix-http-capture-test.sanguine-fe.pages.dev |
Attention: Patch coverage is 9.09091%
with 330 lines
in your changes missing coverage. Please review.
Project coverage is 25.55824%. Comparing base (
8d04213
) to head (661c235
). Report is 10 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description
Summary by CodeRabbit
New Features
Bug Fixes
Refactor
Tests
Chores