Skip to content

fix: point publish.yml at the org App secrets that exist - #5

Merged
vaibhavkumar-deriv merged 1 commit into
masterfrom
fix/use-existing-org-app-secrets
Sep 3, 2026
Merged

fix: point publish.yml at the org App secrets that exist#5
vaibhavkumar-deriv merged 1 commit into
masterfrom
fix/use-existing-org-app-secrets

Conversation

@vaibhavkumar-deriv

Copy link
Copy Markdown
Collaborator

This receiver referenced PROMOTION_APP_ID / PROMOTION_APP_PRIVATE_KEY. Those names were invented by the source repo's scaffold and exist nowhere.

The App already exists org-wide as CLIENT_ID_GHAPP_WRITE / PRIVATE_KEY_GHAPP_WRITE — the same pair deriv-com/deriv-api-schemas uses to drive its own cross-repo publish.yml, with the same actions/create-github-app-token. So no new App and no new secrets are needed; this repo just has to be granted access to them (it currently has no org secrets at all, while the source repo already has these).

Why directly here rather than via promotion: this workflow has to be able to receive the very first dispatch. The scaffold copy only reaches this repo on a promotion that has not run yet — chicken and egg.

The matching change to the scaffold in the source repo is deriv-com/deriv-api-plugin-specs#133, so the two copies stay in step and the next promotion is a no-op on this file.

Two lines plus a header comment. No logic touched. This workflow remains inert until an admin grants the secrets and sets vars.PROMOTION_SOURCE_REPO.

This receiver referenced PROMOTION_APP_ID / PROMOTION_APP_PRIVATE_KEY. Those
names were invented by the source repo's scaffold and exist nowhere.

The App already exists org-wide as CLIENT_ID_GHAPP_WRITE /
PRIVATE_KEY_GHAPP_WRITE — the same pair deriv-com/deriv-api-schemas uses to
drive its own cross-repo publish.yml with the same
actions/create-github-app-token. No new App or secrets are needed; this repo
just has to be granted access to them.

Applied here directly rather than waiting for promotion: this workflow has to
be able to receive the very first dispatch, and the scaffold copy only reaches
this repo on a promotion that has not run yet.

The matching change in the source repo's scaffold is deriv-api-plugin-specs#133,
so the two copies stay in step.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Manifest Files

@vaibhavkumar-deriv
vaibhavkumar-deriv merged commit 2d5f4ef into master Sep 3, 2026
5 checks passed
@vaibhavkumar-deriv
vaibhavkumar-deriv deleted the fix/use-existing-org-app-secrets branch September 3, 2026 16:29
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.

1 participant