feat(redis-cloud): add tracing instrumentation to API client #452
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add #[instrument] spans to all HTTP methods for structured tracing - Add debug! logs for method/URL on each request - Add trace! logs for request bodies and response status - Follows same pattern as redis-enterprise client This establishes a consistent tracing pattern across both API clients for better observability and debugging. Partial fix for #105
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Adds structured tracing instrumentation to the redis-cloud API client, matching the pattern already used in redis-enterprise.
Changes
#[instrument]spans to all HTTP methods (get, post, put, delete, patch)debug!logs for method/URL on each requesttrace!logs for request bodies and response statusBenefits
Related
Partial fix for #105 - establishes the pattern for future tracing enhancements
Closes #105