Apps plugin #254
Open
Apps plugin #254
Conversation
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
- defaultHeaders - createGzipFormData
|
✅ Tests 🎉 All green!❄️ No new flaky tests detected 🔗 Commit SHA: 054f233 | Docs | Datadog PR Page | Was this helpful? Give us feedback! |
yoannmoinet
force-pushed
the
yoann/apps-plugin
branch
from
1cae7b3 to
054f233
Compare
sdkennedy2
approved these changes
Jan 5, 2026
| log.error(red('Unable to determine the git remote to compute the app identifier.')); | ||
| } | ||
|
|
||
| const identifier = buildIdentifier(repository, name); |
There was a problem hiding this comment.
I wonder if we should be worried about conflicts when the repository url isn't defined and the package name conflicts across two apps. I don't think it needs to be solved now but maybe we should think through this scenario in one of the RFCs.
Member
Author
There was a problem hiding this comment.
Very good point, we could make it mandatory to have a remote url.
At least for now.
| const { errors: uploadErrors, warnings: uploadWarnings } = await uploadArchive( | ||
| archive, | ||
| { | ||
| apiKey: context.auth.apiKey, |
There was a problem hiding this comment.
Separately from the PR we should think through how we want to handle auth. I kind of like how dd-auth works.
Member
Author
There was a problem hiding this comment.
How does dd-auth work? (make it through DM in slack)
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.
What and why?
appsplugin that zips built assets and uploads them to Datadog storage with optional dry-run and identifier override.appsplugin into the factory and published bundler packages while keeping alpha plugins hidden from the root README.How?
package.jsonor git remote (with sensitive info stripped)DATADOG_APPS_INTAKE_URL/DATADOG_SITE.createGzipFormDataandgetOriginHeadersfor compressed multipart requests and reuse them in sourcemap sending.hideFromRootReadme.