Replies: 1 comment 1 reply
-
|
We've opened a PR for this: #4389 |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
I’m using dd-trace-go with orchestrion (auto-instrumentation) and noticed that there is currently no way to enable skipRawCommand without using the manual Redis instrumentation API.
In contrib/redis/go-redis.v9/option.go, skipRawCommand can only be configured via code-level options (e.g. WithSkipRawCommand(true)), which works for manual instrumentation but is not accessible when using orchestrion, since Redis clients are auto-instrumented at runtime.
This causes an issue where Redis commands like:
are captured in APM spans (e.g. redis.raw_command or resource names), potentially leaking sensitive information such as tokens, user identifiers, or secrets.
Why this matters
Proposal
Add support for configuring skipRawCommand via an environment variable, for example:
This would:
Happy to help with testing or implementation details if needed.
Thanks for the great work on dd-trace-go!
Beta Was this translation helpful? Give feedback.
All reactions