-
Notifications
You must be signed in to change notification settings - Fork 197
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make sanitize_field_names centrally configurable #856
Conversation
Codecov Report
@@ Coverage Diff @@
## master #856 +/- ##
==========================================
- Coverage 83.83% 83.72% -0.12%
==========================================
Files 141 141
Lines 7849 7863 +14
==========================================
+ Hits 6580 6583 +3
- Misses 863 871 +8
- Partials 406 409 +3
Continue to review full report at Codecov.
|
@axw Thanks for finishing this! Could you also please add the Go agent in Kibana for this central config option: https://github.com/elastic/kibana/blob/caa14d44efedc801fd1f779e15442ca9d5f2fab7/x-pack/plugins/apm/common/agent_configuration/setting_definitions/general_settings.ts#L238 |
Thanks for the reminder @AlexanderWert, I've opened elastic/kibana#85373 |
elastic/apm-agent-go#856 added central config support for 'sanitize_field_names' to the Go agent, so we can now enable it in the UI too.
elastic/apm-agent-go#856 added central config support for 'sanitize_field_names' to the Go agent, so we can now enable it in the UI too.
Closes #821