Skip to content

MOB-6385: Add fed2 endpoint to DatadogSite#2827

Merged
simaoseica-dd merged 2 commits into
developfrom
simaoseica/MOB-6385/fed2-endpoint-
Apr 9, 2026
Merged

MOB-6385: Add fed2 endpoint to DatadogSite#2827
simaoseica-dd merged 2 commits into
developfrom
simaoseica/MOB-6385/fed2-endpoint-

Conversation

@simaoseica-dd

@simaoseica-dd simaoseica-dd commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

What and why?

This PR adds support for a new DatadogSite case, us2_fed, for the FedRAMP-compatible fed2.ddog-gov.com site.

How?

It adds a new case us2_fed to DatadogSite with documentation aligned with the us1_fed case and maps it to its intake endpoint.
It also updates the Objective-C DDSite bridge.

Review checklist

  • Feature or bugfix MUST have appropriate tests (unit, integration)
  • Make sure each commit and the PR mention the Issue number or JIRA reference
  • Add CHANGELOG entry for user facing changes
  • Add Objective-C interface for public APIs - see our guidelines (internal)
  • Run make api-surface when adding new APIs

@simaoseica-dd simaoseica-dd requested review from a team as code owners April 9, 2026 12:12
@simaoseica-dd simaoseica-dd requested a review from alanf April 9, 2026 12:17

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e13c362570

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread api-surface-objc

@maxep maxep left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@alanf alanf left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM!

I will talk to the team more, but I'm not sure if we can utilize this yet, unless we are able to switch the site after the SDK is initialized (since users can theoretically switch between commercial and GOV accounts)

@simaoseica-dd simaoseica-dd merged commit 28a30ff into develop Apr 9, 2026
21 checks passed
@simaoseica-dd simaoseica-dd deleted the simaoseica/MOB-6385/fed2-endpoint- branch April 9, 2026 15:01
@maxep maxep mentioned this pull request Apr 16, 2026
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants