Skip to content

[RUM-6587] mention the Host header in RUM proxy guide #25771

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

BenoitZugmeyer
Copy link
Member

What does this PR do? What is the motivation?

Document how to define the Host header when implementing a RUM data proxy. When this header is kept from the customer's proxy request, the request will fail with SSL errors.

Merge instructions

  • Please merge after reviewing

Additional notes

@BenoitZugmeyer BenoitZugmeyer requested review from a team as code owners October 17, 2024 10:46
@github-actions github-actions bot added the Guide Content impacting a guide label Oct 17, 2024
Copy link
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@@ -19,8 +19,9 @@ To successfully forward a request to Datadog, your proxy must

1. [Build the Datadog intake URL](#build-the-datadog-intake-url).
2. Add an `X-Forwarded-For` header containing the request client IP address for accurate geoIP.
3. Forward the request to the Datadog intake URL using the POST method.
4. Leave the request body unchanged.
3. Remove the `Host` header or set it to Datadog intake URL's host (ex: `browser-intake-datadoghq.com`) (your HTTP client will probably add it automatically).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
3. Remove the `Host` header or set it to Datadog intake URL's host (ex: `browser-intake-datadoghq.com`) (your HTTP client will probably add it automatically).
3. Remove the `Host` header or set it to Datadog intake URL's host, such as `browser-intake-datadoghq.com`.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Guide Content impacting a guide
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants