-
Notifications
You must be signed in to change notification settings - Fork 181
Add OTel for RUM docs #4115
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
base: main
Are you sure you want to change the base?
Add OTel for RUM docs #4115
Conversation
Vale Linting ResultsSummary: 17 suggestions found 💡 Suggestions (17)
|
🔍 Preview links for changed docs |
Co-authored-by: Christoph Heger <christoph.heger@elastic.co>
| proxy_set_header Host $host; | ||
| proxy_set_header X-Real-IP $remote_addr; | ||
| proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for; | ||
| proxy_set_header Authorization $http_authorization; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This config is proxying the Authorization header from the browser most likely in the form ApiKey .... I have my doubts if we want the key being configured in the web app where is public.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@david-luna What's the alternative?
Co-authored-by: David Luna <david.luna@elastic.co>
Summary
Fixes https://github.com/elastic/opentelemetry-dev/issues/1160 by creating a guide to send RUM data to Elastic Observability using OpenTelemetry SDKs and Collectors.
Generative AI disclosure