[APPS] Stop exposing raw index.html standalone URL#329
Merged
gh-worker-dd-mergequeue-cf854d[bot] merged 1 commit intomasterfrom May 6, 2026
Merged
Conversation
Collaborator
Author
This stack of pull requests is managed by Graphite. Learn more about stacking. |
This was referenced Apr 23, 2026
Collaborator
Author
8a8ebf4 to
57382e6
Compare
e1680a9 to
e430250
Compare
|
Codex Review: Didn't find any major issues. Keep them coming! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
sarenji
approved these changes
Apr 23, 2026
michaeldfoley
approved these changes
Apr 23, 2026
yoannmoinet
approved these changes
Apr 24, 2026
e430250 to
3d1a731
Compare
This was referenced May 6, 2026
3d1a731 to
bed9e2f
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Motivation
After uploading a high-code app, the apps plugin was logging two URLs: a raw
index.htmlstandalone URL and the AppBuilder URL. We don't want users to rely on the rawindex.htmlbecause backend functions won't work when the app is loaded that way — backend functions rely on apostMessagebridge to the parent Datadog document, which only exists when the app is rendered inside AppBuilder. Pointing customers at the standalone URL leads them into a broken state.Changes
Dropped the Standalone URL from the post-upload log message. The log now only surfaces the AppBuilder URL, which is the only supported way to run these apps.
Before:
After:
QA Instructions
yarn test:unit packages/plugins/apps/src/upload.test.ts— the existingYour application is available atassertion still passes.Blast Radius
Documentation